Skip to content

feat(settings): improve UX for managing LLM connections #8312

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

Merged
merged 9 commits into from
Aug 5, 2025

Conversation

leoweigand
Copy link
Contributor

@leoweigand leoweigand commented Aug 5, 2025

Important

Improves UX for managing LLM connections by renaming components, adding advanced settings, and refining validation logic in key files.

  • Behavior:
    • Renames 'LLM API key' to 'LLM Connection' in CreateLLMApiKeyDialog.tsx, LLMApiKeyList.tsx, and UpdateLLMApiKeyDialog.tsx.
    • Adds advanced settings toggle in CreateLLMApiKeyForm.tsx for base URL, VertexAI location, extra headers, default models, and custom models.
    • Requires API Base URL for Azure connections in CreateLLMApiKeyForm.tsx.
  • Validation:
    • Updates validation schema in CreateLLMApiKeyForm.tsx to require base URL for Azure and handle Bedrock credentials.
  • UI Components:
    • Adds ChevronDown icon for advanced settings toggle in CreateLLMApiKeyForm.tsx.
    • Replaces PasswordInput with Input for API keys in CreateLLMApiKeyForm.tsx.
  • Misc:
    • Updates button text and dialog titles to reflect new terminology in CreateLLMApiKeyDialog.tsx and UpdateLLMApiKeyDialog.tsx.
    • Adds click handling to prevent event propagation in LLMApiKeyList.tsx.

This description was created by Ellipsis for 5e80fec. You can customize this summary. It will automatically update as commits are pushed.

@leoweigand leoweigand self-assigned this Aug 5, 2025
@dosubot dosubot bot added size:XL This PR changes 500-999 lines, ignoring generated files. feat-api labels Aug 5, 2025
@leoweigand leoweigand added this pull request to the merge queue Aug 5, 2025
@dosubot dosubot bot added the auto-merge This PR is set to be merged label Aug 5, 2025
@leoweigand leoweigand removed this pull request from the merge queue due to a manual request Aug 5, 2025
@dosubot dosubot bot removed the auto-merge This PR is set to be merged label Aug 5, 2025
@leoweigand leoweigand merged commit 90dca15 into main Aug 5, 2025
28 checks passed
@leoweigand leoweigand deleted the leo/lfe-6072 branch August 5, 2025 12:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feat-api size:XL This PR changes 500-999 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant