diff --git a/frontend/src/hooks/query/use-settings.ts b/frontend/src/hooks/query/use-settings.ts index 96a195cceee5..796c0cab1842 100644 --- a/frontend/src/hooks/query/use-settings.ts +++ b/frontend/src/hooks/query/use-settings.ts @@ -27,7 +27,6 @@ export const useSettings = () => { queryKey: ["settings"], queryFn: getSettingsQueryFn, initialData: DEFAULT_SETTINGS, - refetchOnMount: false, }); React.useEffect(() => { diff --git a/frontend/src/services/settings.ts b/frontend/src/services/settings.ts index f838e49e55de..52fd6424cdd1 100644 --- a/frontend/src/services/settings.ts +++ b/frontend/src/services/settings.ts @@ -1,5 +1,3 @@ -import { openHands } from "#/api/open-hands-axios"; - export const LATEST_SETTINGS_VERSION = 5; export type Settings = { @@ -71,32 +69,6 @@ export const getLocalStorageSettings = (): Settings => { }; }; -/** - * Save the settings to the server. Only valid settings are saved. - * @param settings - the settings to save - */ -export const saveSettings = async ( - settings: Partial, -): Promise => { - try { - const apiSettings: Partial = { - llm_model: settings.LLM_MODEL, - llm_base_url: settings.LLM_BASE_URL, - agent: settings.AGENT, - language: settings.LANGUAGE, - confirmation_mode: settings.CONFIRMATION_MODE, - security_analyzer: settings.SECURITY_ANALYZER, - llm_api_key: settings.LLM_API_KEY, - }; - - const { data } = await openHands.post("/api/settings", apiSettings); - return data; - } catch (error) { - // Error handled by returning false - return false; - } -}; - export const maybeMigrateSettings = async (logout: () => void) => { // Sometimes we ship major changes, like a new default agent. // In this case, we may want to override a previous choice made by the user. @@ -120,34 +92,9 @@ export const maybeMigrateSettings = async (logout: () => void) => { if (currentVersion < 4) { logout(); } - - if (currentVersion < 5) { - const localSettings = getLocalStorageSettings(); - await saveSettings(localSettings); - } }; /** * Get the default settings */ export const getDefaultSettings = (): Settings => DEFAULT_SETTINGS; - -/** - * Get the settings from the server or use the default settings if not found - */ -export const getSettings = async (): Promise => { - const { data: apiSettings } = - await openHands.get("/api/settings"); - if (apiSettings != null) { - return { - LLM_MODEL: apiSettings.llm_model, - LLM_BASE_URL: apiSettings.llm_base_url, - AGENT: apiSettings.agent, - LANGUAGE: apiSettings.language, - CONFIRMATION_MODE: apiSettings.confirmation_mode, - SECURITY_ANALYZER: apiSettings.security_analyzer, - LLM_API_KEY: "", - }; - } - return getLocalStorageSettings(); -};