From a923d24f274817ca6abb8fcf7251c5483f2bc754 Mon Sep 17 00:00:00 2001 From: Lily Delalande <119957291+lily-de@users.noreply.github.com> Date: Wed, 2 Apr 2025 16:21:03 -0400 Subject: [PATCH] Fix function params (#2012) --- .../models/model_list/BaseModelsList.tsx | 14 +++++++++----- 1 file changed, 9 insertions(+), 5 deletions(-) diff --git a/ui/desktop/src/components/settings_v2/models/model_list/BaseModelsList.tsx b/ui/desktop/src/components/settings_v2/models/model_list/BaseModelsList.tsx index 789a9bc2..85e0641b 100644 --- a/ui/desktop/src/components/settings_v2/models/model_list/BaseModelsList.tsx +++ b/ui/desktop/src/components/settings_v2/models/model_list/BaseModelsList.tsx @@ -1,8 +1,9 @@ import React, { useEffect, useState } from 'react'; -import Model, { getProviderMetadata } from '../modelInterface'; +import Model from '../modelInterface'; import { useRecentModels } from './recentModels'; import { changeModel, getCurrentModelAndProvider } from '../index'; import { useConfig } from '../../../ConfigContext'; +import { toastInfo } from '../../../../toasts'; interface ModelRadioListProps { renderItem: (props: { @@ -29,7 +30,7 @@ export function BaseModelsList({ } else { modelList = providedModelList; } - const { read, upsert, getProviders } = useConfig(); + const { read, upsert, getExtensions, addExtension } = useConfig(); const [selectedModel, setSelectedModel] = useState(null); const [isInitialized, setIsInitialized] = useState(false); @@ -71,8 +72,7 @@ export function BaseModelsList({ }, [read]); const handleModelSelection = async (model: Model) => { - // Fix: Use the model parameter that's passed in - await changeModel({ model: model, writeToConfig: upsert }); + await changeModel({ model: model, writeToConfig: upsert, getExtensions, addExtension }); }; // Updated to work with CustomRadio @@ -83,7 +83,11 @@ export function BaseModelsList({ selectedModel.name === model.name && selectedModel.provider === model.provider ) { - console.log(`Model "${model.name}" is already active.`); + toastInfo({ + title: 'No change', + msg: `Model "${model.name}" is already active.`, + }); + return; }