Refactor to always hide modal

The delete confirm modal would still be visible on error. This always hides the modal
elastic · Dec 3, 2021 · c6c06f3 · c6c06f3
1 parent e52e145
commit c6c06f3
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/...s/enterprise_search/public/applications/workplace_search/views/api_keys/api_keys_logic.ts b/...s/enterprise_search/public/applications/workplace_search/views/api_keys/api_keys_logic.ts
@@ -136,15 +136,13 @@ export const ApiKeysLogic = kea<MakeLogicType<ApiKeysLogicValues, ApiKeysLogicAc
       {
         showDeleteModal: () => true,
         hideDeleteModal: () => false,
-        fetchApiKeys: () => false,
       },
     ],
     apiTokenNameToDelete: [
       '',
       {
         showDeleteModal: (_, tokenName) => tokenName,
         hideDeleteModal: () => '',
-        fetchApiKeys: () => '',
       },
     ],
     formErrors: [
@@ -189,6 +187,8 @@ export const ApiKeysLogic = kea<MakeLogicType<ApiKeysLogicValues, ApiKeysLogicAc
         flashSuccessToast(DELETE_MESSAGE(apiTokenNameToDelete));
       } catch (e) {
         flashAPIErrors(e);
+      } finally {
+        actions.hideDeleteModal();
       }
     },
     onApiFormSubmit: async () => {