This is an automated email from the ASF dual-hosted git repository.
michaelsmolina pushed a commit to branch 3.1
in repository https://gitbox.apache.org/repos/asf/superset.git
The following commit(s) were added to refs/heads/3.1 by this push:
new 6cdaf479f2 fix(sqllab): autosync fail on migrated queryEditor (#26922)
6cdaf479f2 is described below
commit 6cdaf479f23baac6fada36409aca43bf44e7d8f4
Author: JUST.in DO IT <[email protected]>
AuthorDate: Wed Jan 31 11:19:55 2024 -0800
fix(sqllab): autosync fail on migrated queryEditor (#26922)
---
superset-frontend/src/SqlLab/actions/sqlLab.js | 1 +
superset-frontend/src/SqlLab/actions/sqlLab.test.js | 11 ++++++++---
2 files changed, 9 insertions(+), 3 deletions(-)
diff --git a/superset-frontend/src/SqlLab/actions/sqlLab.js
b/superset-frontend/src/SqlLab/actions/sqlLab.js
index 567d3383d7..97db64fb89 100644
--- a/superset-frontend/src/SqlLab/actions/sqlLab.js
+++ b/superset-frontend/src/SqlLab/actions/sqlLab.js
@@ -476,6 +476,7 @@ export function migrateQueryEditorFromLocalStorage(
const newQueryEditor = {
...queryEditor,
id: json.id.toString(),
+ inLocalStorage: false,
};
dispatch({
type: MIGRATE_QUERY_EDITOR,
diff --git a/superset-frontend/src/SqlLab/actions/sqlLab.test.js
b/superset-frontend/src/SqlLab/actions/sqlLab.test.js
index 175ea06ec3..dd48ed8c7b 100644
--- a/superset-frontend/src/SqlLab/actions/sqlLab.test.js
+++ b/superset-frontend/src/SqlLab/actions/sqlLab.test.js
@@ -937,12 +937,17 @@ describe('async actions', () => {
{ ...query, id: 'previewTwo' },
];
const store = mockStore({});
+ const oldQueryEditor = { ...queryEditor, inLocalStorage: true };
const expectedActions = [
{
type: actions.MIGRATE_QUERY_EDITOR,
- oldQueryEditor: queryEditor,
+ oldQueryEditor,
// new qe has a different id
- newQueryEditor: { ...queryEditor, id: '1' },
+ newQueryEditor: {
+ ...oldQueryEditor,
+ id: '1',
+ inLocalStorage: false,
+ },
},
{
type: actions.MIGRATE_TAB_HISTORY,
@@ -975,7 +980,7 @@ describe('async actions', () => {
return store
.dispatch(
actions.migrateQueryEditorFromLocalStorage(
- queryEditor,
+ oldQueryEditor,
tables,
queries,
),
