fix: Dashboard import holding issue

superset-frontend/src/views/CRUD/hooks.ts

@@ -448,7 +448,10 @@ export function useImportResource(
                 t(
                   'An error occurred while importing %s: %s',
                   resourceLabel,
-                  error.errors.map(payload => payload.message).join('\n'),
+                  [
+                    ...error.errors.map(payload => payload.message),
+                    'Please re-export your file and try importing again',
+                  ].join('\n'),
                 ),
               );
             } else {
@@ -464,7 +467,7 @@ export function useImportResource(
           updateState({ loading: false });
         });
     },
-    [],
+    [handleErrorMsg, resourceLabel, resourceName],
   );
 
   return { state, importResource };

superset-frontend/src/views/CRUD/utils.test.tsx

@@ -46,6 +46,25 @@ const terminalErrors = {
   ],
 };
 
+const terminalErrorsWithOnlyIssuesCode = {
+  errors: [
+    {
+      message: 'Error importing database',
+      error_type: 'GENERIC_COMMAND_ERROR',
+      level: 'warning',
+      extra: {
+        issue_codes: [
+          {
+            code: 1010,
+            message:
+              'Issue 1010 - Superset encountered an error while running a command.',
+          },
+        ],
+      },
+    },
+  ],
+};
+
 const overwriteNeededErrors = {
   errors: [
     {
@@ -146,6 +165,12 @@ test('detects if the error message is terminal or if it requires uses interventi
   expect(isTerminal).toBe(false);
 });
 
+test('error message is terminal when the extra field not contains other keys excepting issue_codes', () => {
+  expect(hasTerminalValidation(terminalErrorsWithOnlyIssuesCode.errors)).toBe(
+    true,
+  );
+});
+
 test('does not ask for password when the import type is wrong', () => {
   const error = {
     errors: [

superset-frontend/src/views/CRUD/utils.tsx

@@ -401,15 +401,17 @@ export const getAlreadyExists = (errors: Record<string, any>[]) =>
     .flat();
 
 export const hasTerminalValidation = (errors: Record<string, any>[]) =>
-  errors.some(
-    error =>
-      !Object.entries(error.extra)
-        .filter(([key, _]) => key !== 'issue_codes')
-        .every(
-          ([_, payload]) =>
-            isNeedsPassword(payload) || isAlreadyExists(payload),
-        ),
-  );
+  errors.some(error => {
+    const not_issues_codes = Object.entries(error.extra).filter(
+      ([key, _]) => key !== 'issue_codes',
+    );
+
+    if (not_issues_codes.length === 0) return true;
+
+    return !not_issues_codes.every(
+      ([_, payload]) => isNeedsPassword(payload) || isAlreadyExists(payload),
+    );
+  });
 
 export const checkUploadExtensions = (
   perm: Array<any> | string | undefined | boolean,