Skip to content

Commit

Permalink
Fix expected field names in type in health check details
Browse files Browse the repository at this point in the history
The previous PR was merged a bit too early (sorry, I should have marked
it as draft). This one fixes up the field names after a bit more testing.
  • Loading branch information
sd2k committed Sep 27, 2023
1 parent 45ec9cb commit 4a727a9
Show file tree
Hide file tree
Showing 3 changed files with 4 additions and 4 deletions.
2 changes: 1 addition & 1 deletion src/llms/openai.ts
Original file line number Diff line number Diff line change
Expand Up @@ -351,5 +351,5 @@ export const enabled = async () => {
setLLMPluginVersion(details.version);
}
// If the plugin is installed then check if it is configured.
return details?.openAIEnabled ?? false;
return details?.openAI ?? false;
}
4 changes: 2 additions & 2 deletions src/llms/types.ts
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
export type LLMAppHealthCheck = {
details: {
openAIEnabled?: boolean;
vectorEnabled?: boolean;
openAI?: boolean;
vector?: boolean;
version?: string;
};
};
2 changes: 1 addition & 1 deletion src/llms/vector.ts
Original file line number Diff line number Diff line change
Expand Up @@ -95,5 +95,5 @@ export const enabled = async () => {
setLLMPluginVersion(details.version);
}
// If the plugin is installed then check if it is configured.
return details.vectorEnabled ?? false;
return details.vector ?? false;
};

0 comments on commit 4a727a9

Please sign in to comment.