Skip to content

Commit 8ee185c

Browse files
authored
remove the required asterisk from the base url & fix cross-config problems (#217)
* remove the required asterisk from the base url * clean out other configs when using open ai and vice versa
1 parent 7f9e0cd commit 8ee185c

File tree

2 files changed

+6
-1
lines changed

2 files changed

+6
-1
lines changed

ui/src/pages/Settings/ModelProviderFields.tsx

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -117,7 +117,6 @@ export const ModelProviderFields = ({
117117
label={"OpenAI Base URL"}
118118
initialValue={projectConfig?.openai_config.openai_api_base}
119119
name={["openai_config", "openai_api_base"]}
120-
required={modelProvider === "OpenAI"}
121120
tooltip="The base URL for the OpenAI service."
122121
hidden={modelProvider !== "OpenAI"}
123122
>

ui/src/pages/Settings/RestartAppModal.tsx

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -166,11 +166,17 @@ const RestartAppModal = ({
166166
.then((values) => {
167167
if (modelProvider === "CAII") {
168168
values.azure_config = {};
169+
values.openai_config = {};
169170
} else if (modelProvider === "Bedrock") {
170171
values.azure_config = {};
171172
values.caii_config = {};
173+
values.openai_config = {};
172174
} else if (modelProvider === "Azure") {
173175
values.caii_config = {};
176+
values.openai_config = {};
177+
} else if (modelProvider === "OpenAI") {
178+
values.azure_config = {};
179+
values.caii_config = {};
174180
}
175181

176182
if (selectedFileStorage === "Local") {

0 commit comments

Comments
 (0)