-
Notifications
You must be signed in to change notification settings - Fork 8
Add dashboard-managed LLM provider system with OpenAI-compatible API support #13
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Changes from all commits
e996a54
ff77850
0293b36
5eb8c1a
04372bc
426bda0
1b58039
825e2f1
b49a816
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change | ||||||||||||
|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|
|
|
@@ -36,7 +36,8 @@ cp .dev.vars.example .dev.vars | |||||||||||||
| You will need to set up: | ||||||||||||||
| - A GitHub App (for webhooks/checks). | ||||||||||||||
| - A GitHub OAuth App (for dashboard authentication). | ||||||||||||||
| - A Gemini API Key. | ||||||||||||||
| - `LLM_CONFIG_ENCRYPTION_KEY` for encrypting dashboard-managed provider API keys. | ||||||||||||||
|
devarshishimpi marked this conversation as resolved.
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. The newly added lines for 'LLM_CONFIG_ENCRYPTION_KEY' and 'LLM providers' are missing the leading bullet point ('- ') used by the other items in the setup list, which breaks the document's visual consistency.
Suggested change
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. The newly added lines lack the bullet point prefix ('- ') used by the other items in the setup list, which breaks the visual consistency of the documentation.
Suggested change
|
||||||||||||||
| - LLM providers and model credentials from the Settings dashboard. | ||||||||||||||
| - A Hyperdrive local connection string for `wrangler dev`. | ||||||||||||||
| - A direct `DATABASE_URL` for migrations. | ||||||||||||||
|
|
||||||||||||||
|
|
||||||||||||||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The newly added items for LLM configuration are missing the bullet point prefix ('- ') used in the preceding lines (lines 37-38). This breaks the Markdown list structure and results in inconsistent visual formatting in the rendered document.