-
Notifications
You must be signed in to change notification settings - Fork 14.1k
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
SQLLab - Saving a query with the same label results in a new query being created #5804
Comments
I am taking this one. More complex than it is seems. No provision for REST update call in the component, React components not receiving the id (and additionally the description). This causes the create to always occur which explains the dup. On it. |
I have fixed this bug and added all appropriate tests. However, it is necessary to change superset ui connection. See this link for more info -> apache-superset/superset-ui#104. I have submitted this ticket and a PR with the fix. Will need to wait for this fix to be applied there in order to send a PR for this one. As soon as applied there, I will send a PR for this fix. |
pull request apache-superset/superset-ui#105 created in superset ui as a dependency to this problem |
… results in a new query being created
… results in a new query being created
… results in a new query being created
This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions. For admin, please label this issue |
Make sure these boxes are checked before submitting your issue - thank you!
Superset version
0.25.6
Expected results
When you open a saved query in SQLLab and then save it with the same label, the existing query is updated.
Actual results
When you open a saved query in SQLLab and then save it with the same label, a new query is created.
Steps to reproduce
Results:
There are two queries
test1
saved queries listed.The text was updated successfully, but these errors were encountered: