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
fix: save tabs when saving the query bug #12607
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
}) | ||
.then(() => { | ||
dispatch(addSuccessToast(t('Your query was updated'))); | ||
dispatch(queryEditorSetTitle(query, query.title)); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This line is the only difference, right?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
yup, but moved it to the bottom of the page
Codecov Report
@@ Coverage Diff @@
## master #12607 +/- ##
==========================================
- Coverage 66.72% 63.06% -3.67%
==========================================
Files 1014 1015 +1
Lines 49612 50929 +1317
Branches 4839 4839
==========================================
- Hits 33106 32117 -989
- Misses 16383 18614 +2231
- Partials 123 198 +75
Flags with carried forward coverage won't be shown. Click here to find out more.
Continue to review full report at Codecov.
|
SUMMARY
Fix small bug with updating the query tab name on query save. This bug only appears specifically when SQLLAB_BACKEND_PERSISTENCE is True.
Reference for #12561
BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF
TEST PLAN
SQLLAB_BACKEND_PERSISTENCE
is True insuperset_config.py
ADDITIONAL INFORMATION