Skip to content

Add deduplication checkbox#143

Draft
sverhoeven wants to merge 2 commits into
mainfrom
de-dup-81
Draft

Add deduplication checkbox#143
sverhoeven wants to merge 2 commits into
mainfrom
de-dup-81

Conversation

@sverhoeven

Copy link
Copy Markdown
Collaborator

Default with de-duplicated mode off
localhost_5173__project=Example (3)

With de-duplicated mode on
localhost_5173__project=Example (2)

Fixes #81

TODO

  • handle inline TODOs
  • when in deduplicated mode, make sure editing/uploading/restore from local storage still works
  • check if it is ok to do de-duplication on all components not just Activation Condition component.

@sverhoeven

Copy link
Copy Markdown
Collaborator Author

Turning on de-duplication, causes difficulties when you make changes, for example changing text of a hidden activation condition will need to unhide a component node. This requires if statements where the statements/connectons are converted to a graph.

Maybe it is easier to have separate view in a modal dialog that prevents editing and the views graph can be generated on dialog open.

@sverhoeven sverhoeven changed the base branch from export-network to main March 19, 2025 06:51
@kinsta

kinsta Bot commented Apr 1, 2025

Copy link
Copy Markdown

Preview deployments for INA-tool ⚡️

Status Branch preview Commit preview
✅ Ready Visit preview Visit preview

Commit: 886ce80d7608da0c3433acdaa8d55c8f9cf1cf54

Deployment ID: bac08a6b-e2ad-4a33-8d1b-e60aac34254a

Static site name: ina-tool-5f19y

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[NEW] Let activation condition be shared between multiple statements

1 participant