Skip to content

Commit

Permalink
fix: do not retry for reveal 404, will clear existing form entries
Browse files Browse the repository at this point in the history
  • Loading branch information
ivyjeong13 committed Dec 17, 2024
1 parent d889f28 commit c62bfc6
Showing 1 changed file with 5 additions and 1 deletion.
Original file line number Diff line number Diff line change
Expand Up @@ -127,7 +127,11 @@ export function ModelProviderConfigureContent({
ModelProviderApiService.revealModelProviderById.key(modelProvider.id),
({ modelProviderId }) =>
ModelProviderApiService.revealModelProviderById(modelProviderId),
{ keepPreviousData: true }
{
keepPreviousData: true,
// 404 -> no credential found, so don't retry
shouldRetryOnError: (error) => error.status !== 404,
}
);

const requiredParameters = modelProvider.requiredConfigurationParameters;
Expand Down

0 comments on commit c62bfc6

Please sign in to comment.