diff --git a/backend/danswer/danswerbot/slack/handlers/handle_regular_answer.py b/backend/danswer/danswerbot/slack/handlers/handle_regular_answer.py
index f1c9bd077cf0..5bd920c4b6cb 100644
--- a/backend/danswer/danswerbot/slack/handlers/handle_regular_answer.py
+++ b/backend/danswer/danswerbot/slack/handlers/handle_regular_answer.py
@@ -160,7 +160,7 @@ def handle_regular_answer(
detail="Slack bot does not support persona config",
)
- elif new_message_request.persona_id:
+ elif new_message_request.persona_id is not None:
persona = cast(
Persona,
fetch_persona_by_id(
diff --git a/web/src/app/admin/assistants/PersonaTable.tsx b/web/src/app/admin/assistants/PersonaTable.tsx
index e30e858f13c1..e1e871372852 100644
--- a/web/src/app/admin/assistants/PersonaTable.tsx
+++ b/web/src/app/admin/assistants/PersonaTable.tsx
@@ -119,7 +119,7 @@ export function PersonasTable({
id: persona.id.toString(),
cells: [
- {!persona.is_default_persona && (
+ {!persona.builtin_persona && (
diff --git a/web/src/app/chat/ChatPage.tsx b/web/src/app/chat/ChatPage.tsx
index 0d13b8607d28..e9676d282469 100644
--- a/web/src/app/chat/ChatPage.tsx
+++ b/web/src/app/chat/ChatPage.tsx
@@ -208,7 +208,7 @@ export function ChatPage({
};
const llmOverrideManager = useLlmOverride(
- user?.preferences.default_model,
+ user?.preferences.default_model ?? null,
selectedChatSession,
defaultTemperature
);
@@ -1779,6 +1779,7 @@ export function ChatPage({
{settingsToggled && (
void;
llmProviders: LLMProviderDescriptor[];
setLlmOverride: Dispatch>;
onClose: () => void;
defaultModel: string | null;
refreshUser: () => void;
}) {
- const { popup, setPopup } = usePopup();
const containerRef = useRef(null);
const messageRef = useRef(null);
@@ -127,7 +128,6 @@ export function SetDefaultModelModal({
modalClassName="rounded-lg bg-white max-w-xl"
>
<>
- {popup}
Set Default Model
diff --git a/web/src/lib/hooks.ts b/web/src/lib/hooks.ts
index 39613e388fdf..a760c0824715 100644
--- a/web/src/lib/hooks.ts
+++ b/web/src/lib/hooks.ts
@@ -151,7 +151,7 @@ export function useLlmOverride(
defaultTemperature?: number
): LlmOverrideManager {
const [globalDefault, setGlobalDefault] = useState(
- globalModel
+ globalModel != null
? destructureValue(globalModel)
: {
name: "",
@@ -182,6 +182,18 @@ export function useLlmOverride(
defaultTemperature != undefined ? defaultTemperature : 0
);
+ useEffect(() => {
+ setGlobalDefault(
+ globalModel != null
+ ? destructureValue(globalModel)
+ : {
+ name: "",
+ provider: "",
+ modelName: "",
+ }
+ );
+ }, [globalModel]);
+
useEffect(() => {
setTemperature(defaultTemperature !== undefined ? defaultTemperature : 0);
}, [defaultTemperature]);