Warn lack of API credential when creating assistant

chieapp · Jul 30, 2023 · 6cfe7cc · 6cfe7cc
1 parent eb90eb1
commit 6cfe7cc
Show file tree

Hide file tree

Showing 3 changed files with 12 additions and 2 deletions.
diff --git a/src/controller/window-manager.ts b/src/controller/window-manager.ts
@@ -3,6 +3,8 @@ import gui from 'gui';
 import Assistant from '../model/assistant';
 import BaseWindow from '../view/base-window';
 import WindowStore, {WindowStoreData} from '../model/window-store';
+import alert from '../util/alert';
+import apiManager from '../controller/api-manager';
 import assistantManager from './assistant-manager';
 import {collectGarbage} from './gc-center';
 import {ConfigStoreItem} from '../model/config-store';
@@ -77,6 +79,14 @@ export class WindowManager extends ConfigStoreItem {
     return this.#namedWindows.showWindow(name);
   }
 
+  showNewAssistantWindowOrError() {
+    const hasCredential = apiManager.getCredentials().length > 0;
+    if (hasCredential)
+      this.showNamedWindow('newAssistant');
+    else
+      alert('Can not create new assistant before adding API credentials.');
+  }
+
   getCurrentWindow() {
     return this.windows.find(w => w.window.isActive());
   }

diff --git a/src/view/base-menu-bar.ts b/src/view/base-menu-bar.ts
@@ -132,7 +132,7 @@ export default class BaseMenuBar extends SignalsOwner {
         },
         {
           label: 'New Assistant...',
-          onClick: () => windowManager.showNamedWindow('newAssistant'),
+          onClick: () => windowManager.showNewAssistantWindowOrError(),
         },
         { type: 'separator' },
       ],

diff --git a/src/view/dashboard-window.ts b/src/view/dashboard-window.ts
@@ -66,7 +66,7 @@ export default class DashboardWindow extends BaseWindow {
 
     this.#addButton = new IconButton('add');
     this.#addButton.view.setTooltip('Add new assistant');
-    this.#addButton.onClick = () => windowManager.showNamedWindow('newAssistant');
+    this.#addButton.onClick = () => windowManager.showNewAssistantWindowOrError();
     this.#sidebar.view.addChildView(this.#addButton.view);
 
     // Create views for assistants.