Merge pull request #4575 from BerriAI/ui_fix_errors_when_no_model_list_defined

UI - don't spam error messages when model list is not defined
This commit is contained in:
Ishaan Jaff 2024-07-06 14:38:02 -07:00 committed by GitHub
commit e34ba00a51
No known key found for this signature in database
GPG key ID: B5690EEEBB952194
2 changed files with 18 additions and 4 deletions

View file

@ -701,6 +701,9 @@ export const claimOnboardingToken = async (
throw error;
}
};
let ModelListerrorShown = false;
let errorTimer: NodeJS.Timeout | null = null;
export const modelInfoCall = async (
accessToken: String,
userID: String,
@ -722,8 +725,21 @@ export const modelInfoCall = async (
});
if (!response.ok) {
const errorData = await response.text();
message.error(errorData, 10);
let errorData = await response.text();
errorData += `error shown=${ModelListerrorShown}`
if (!ModelListerrorShown) {
if (errorData.includes("No model list passed")) {
errorData = "No Models Exist. Click Add Model to get started.";
}
message.info(errorData, 10);
ModelListerrorShown = true;
if (errorTimer) clearTimeout(errorTimer);
errorTimer = setTimeout(() => {
ModelListerrorShown = false;
}, 10000);
}
throw new Error("Network response was not ok");
}
@ -758,7 +774,6 @@ export const modelHubCall = async (accessToken: String) => {
if (!response.ok) {
const errorData = await response.text();
message.error(errorData, 10);
throw new Error("Network response was not ok");
}

View file

@ -32,7 +32,6 @@ import {
allTagNamesCall,
modelMetricsCall,
modelAvailableCall,
modelInfoCall,
adminspendByProvider,
adminGlobalActivity,
adminGlobalActivityPerModel,