apply review feedback

ansible · Oct 11, 2024 · ea7b5a8 · ea7b5a8
1 parent 538ef2b
commit ea7b5a8
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 6 deletions.
diff --git a/ansible_ai_connect_admin_portal/config/webpack.mock.api.keys.modelids.ts b/ansible_ai_connect_admin_portal/config/webpack.mock.api.keys.modelids.ts
@@ -73,12 +73,10 @@ export default webpackMockServer.add((app, helper) => {
       keys.delete(ORG_ID);
       if (modelIds.has(ORG_ID)) {
         modelIds.delete(ORG_ID);
-      } else {
-        res.sendStatus(404);
       }
       res.sendStatus(200);
     } else {
-      res.sendStatus(404);
+      res.sendStatus(400);
     }
   });
 

diff --git a/ansible_ai_connect_admin_portal/src/ModelSettingsKeyDeletionModal.tsx b/ansible_ai_connect_admin_portal/src/ModelSettingsKeyDeletionModal.tsx
@@ -59,15 +59,15 @@ export const ModelSettingsKeyDeletionModal = (
       onClose={handleModalToggle}
       data-testid={"model-settings-key-deletion-modal"}
       actions={[
-        <BusyButton
+        <Button
           key="delete"
           variant="danger"
           onClick={deleteKey}
-          isBusy={isDeleting}
+          isDisabled={isDeleting}
           data-testid={"model-settings-key-deletion-modal__delete-button"}
         >
           {t("Delete")}
-        </BusyButton>,
+        </Button>,
         <Button key="cancel" variant="link" onClick={handleCancel}>
           {t("Cancel")}
         </Button>,