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

Saved Queries save modal closes if no changes were made on that tab #31767

Open
3 tasks done
msyavuz opened this issue Jan 9, 2025 · 2 comments
Open
3 tasks done

Saved Queries save modal closes if no changes were made on that tab #31767

msyavuz opened this issue Jan 9, 2025 · 2 comments
Assignees

Comments

@msyavuz
Copy link
Contributor

msyavuz commented Jan 9, 2025

Bug description

  1. Go to saved queries
  2. Click add query
  3. Add a new tab and click save
  4. Save modal disappears if no changes were made to the query

Screenshots/recordings

No response

Superset version

master / latest-dev

Python version

3.9

Node version

16

Browser

Chrome

Additional context

No response

Checklist

  • I have searched Superset docs and Slack and didn't find a solution to my problem.
  • I have searched the GitHub issue tracker and didn't find a similar bug report.
  • I have checked Superset's logs for errors and if I found a relevant Python stacktrace, I included it here as text in the "additional context" section.
Copy link

dosubot bot commented Jan 9, 2025

The behavior you're experiencing, where the save modal closes if no changes are made to a query in a new tab, might be related to how the SaveQuery component handles the save action. The modal is designed to close when the onSaveWrapper function is called, which includes logging the save action and then closing the modal. However, if no changes are detected, the modal might be closing prematurely due to the logic that determines whether a save action is necessary [1].

Unfortunately, I couldn't find specific details in the repository about how changes are detected in a new tab or if there's a condition that prevents the modal from closing when no changes are made. It might be worth investigating the logic in the SaveQuery component further to see if there's a condition or flag that could be adjusted to prevent the modal from closing in this scenario.

To continue talking to Dosu, mention @dosu.


Help Dosu learn! Give it feedback: Great Response | Irrelevant Answer | Incorrect Sources | Too Verbose | Hallucination | Other

@alexandrusoare
Copy link
Contributor

Hey I would like to solve this issue

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

No branches or pull requests

2 participants