Update spread order (#2386)

* update spread

* update
This commit is contained in:
pablodanswer
2024-09-10 11:04:47 -07:00
committed by GitHub
parent f1c5e80f17
commit 07f49a384f

View File

@@ -191,12 +191,13 @@ export default function EmbeddingForm() {
} }
let newModel: SavedSearchSettings; let newModel: SavedSearchSettings;
// We use a spread operation to merge properties from multiple objects into a single object.
// Advanced embedding details may update default values.
if (selectedProvider.provider_type != null) { if (selectedProvider.provider_type != null) {
// This is a cloud model // This is a cloud model
newModel = { newModel = {
...advancedEmbeddingDetails,
...selectedProvider,
...rerankingDetails, ...rerankingDetails,
...advancedEmbeddingDetails,
model_name: selectedProvider.model_name, model_name: selectedProvider.model_name,
provider_type: provider_type:
(selectedProvider.provider_type (selectedProvider.provider_type
@@ -206,9 +207,9 @@ export default function EmbeddingForm() {
} else { } else {
// This is a locally hosted model // This is a locally hosted model
newModel = { newModel = {
...advancedEmbeddingDetails,
...selectedProvider, ...selectedProvider,
...rerankingDetails, ...rerankingDetails,
...advancedEmbeddingDetails,
model_name: selectedProvider.model_name!, model_name: selectedProvider.model_name!,
provider_type: null, provider_type: null,
}; };
@@ -225,6 +226,7 @@ export default function EmbeddingForm() {
}, },
} }
); );
if (response.ok) { if (response.ok) {
setPopup({ setPopup({
message: "Changed provider suceessfully. Redirecing to embedding page", message: "Changed provider suceessfully. Redirecing to embedding page",