Merge pull request #218 from keen/Bugfix/#206

Fix for update access key
keen · Dec 9, 2019 · d069ff3 · d069ff3
2 parents 768d1aa + f56e4c5
commit d069ff3
Showing 1 changed file with 10 additions and 4 deletions.
diff --git a/lib/actions/rootActions.js b/lib/actions/rootActions.js
@@ -246,7 +246,7 @@ export const selectSavedQuery = (queryNames, index) => (dispatch, getState) => {
   const isPublic = dashboardInfo.is_public;
   if (isPublic && savedQueries.length) {
     const isNewQueryAdded = savedQueries.some(
-      query => !savedQueriesList.includes(query)
+      query => !savedQueriesList.includes(query.value)
     );
     if (isNewQueryAdded) {
       updateAPIKey(newSavedQueriesList, dashboardInfo.title);
@@ -304,9 +304,15 @@ export const deleteChart = index => (dispatch, getState) => {
     dispatch(changeSavedQueryList(delValue, []));
     const newStore = getState();
     const savedQueriesList = getSavedQueriesList(newStore);
-    delValue.some(sq => !savedQueriesList.includes(sq)) &&
-      dashboardInfo.is_public &&
-      updateAPIKey(savedQueriesList, dashboardInfo.title);
+    const isPublic = dashboardInfo.is_public;
+    if (isPublic && delValue.length) {
+      const isNewQueryAdded = delValue.some(
+        query => !savedQueriesList.includes(query.value)
+      );
+      if (isNewQueryAdded) {
+        updateAPIKey(savedQueriesList, dashboardInfo.title);
+      }
+    }
     dispatch({
       type: actions.DELETE_CHART,
       index