From a34a96a5ffcea8e4168869a585823f0d6d1e7936 Mon Sep 17 00:00:00 2001
From: mic <m@micz.it>
Date: Mon, 29 Jul 2024 23:47:14 +0200
Subject: [PATCH] correctly managing apikey and model in the option page

---
 options/mzta-options.js | 7 ++++---
 1 file changed, 4 insertions(+), 3 deletions(-)

diff --git a/options/mzta-options.js b/options/mzta-options.js
index df03ed5f..cfe00669 100644
--- a/options/mzta-options.js
+++ b/options/mzta-options.js
@@ -140,7 +140,6 @@ document.addEventListener('DOMContentLoaded', async () => {
   });
 
   showConnectionOptions();
-  warnAPIKeyEmpty();
   
   document.getElementById('btnManagePrompts').addEventListener('click', () => {
     // check if the tab is already there
@@ -161,7 +160,6 @@ document.addEventListener('DOMContentLoaded', async () => {
   document.getElementById("chatgpt_api_key").addEventListener("change", warnAPIKeyEmpty);
 
   let prefs = await browser.storage.sync.get({chatgpt_api_key: '', chatgpt_model: ''});
-  let openai = new OpenAI(prefs.chatgpt_api_key, true);
   let select_chatgpt_model = document.getElementById('chatgpt_model');
 
   const option = document.createElement('option');
@@ -169,8 +167,9 @@ document.addEventListener('DOMContentLoaded', async () => {
   option.text = prefs.chatgpt_model;
   select_chatgpt_model.appendChild(option);
 
-  document.getElementById('btnUpdateChatGPTModels').addEventListener('click', () => {
+  document.getElementById('btnUpdateChatGPTModels').addEventListener('click', async () => {
     document.getElementById('model_fetch_loading').style.display = 'inline';
+    let openai = new OpenAI(document.getElementById("chatgpt_api_key").value, true);
     openai.fetchModels().then((data) => {
       if(!data){
         document.getElementById('model_fetch_loading').style.display = 'none';
@@ -189,5 +188,7 @@ document.addEventListener('DOMContentLoaded', async () => {
       });
       document.getElementById('model_fetch_loading').style.display = 'none';
     })
+    
+    warnAPIKeyEmpty();
   });
 }, { once: true });