Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Insight Editor Saving Issues #236

Closed
Jeehut opened this issue Apr 28, 2023 · 2 comments
Closed

Insight Editor Saving Issues #236

Jeehut opened this issue Apr 28, 2023 · 2 comments
Labels
bug Something isn't working

Comments

@Jeehut
Copy link

Jeehut commented Apr 28, 2023

I'm having an issue while editing an insiqht.

Insight ID: 371B712F-1AFD-49FF-9B66-6C28181513FA

When I duplicate an insight, the duplicated insight behaves weirdly. For example, when I change the title, it doesn't save on first try. I have to do it a second time to change it. Or when I edit the value of a filter, it doesn't save, instead it creates a new filter but keeps the old and combines them automatically. Even if I explicitly delete the old, it won't save. It's weird.

@winsmith
Copy link
Contributor

winsmith commented May 2, 2023

Thanks for reporting, the root cause for this is probably the same as #217, i.e. you can somehow either change metadata or actual data, because they are managed by different components and there's a sync issue.

@winsmith winsmith added the bug Something isn't working label May 2, 2023
@winsmith winsmith changed the title Editing duplicated insights behaves weirdly Insight Editor Saving Issues Oct 19, 2023
@winsmith
Copy link
Contributor

winsmith commented Oct 20, 2023

To Do:

  • Write various tests for InsightEditor to make sure it continues working as expected
  • Explicitly test duplicating insights
  • Update InsightEditor to return not the full model but only updated keys and values
  • Update InsightEditor to no longer track values internally but always display the values passed in from the outside
  • Use the controller's delayedSave method to save the insight

@winsmith winsmith closed this as completed Nov 9, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants