Skip to content
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

fix: update LocalProvidersTab.tsx #1362

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

Stijnus
Copy link
Collaborator

@Stijnus Stijnus commented Feb 22, 2025

Local Provider Settings Enhancement

Changes Made

  • Added dynamic URL configuration support for local providers (Ollama, LMStudio, OpenAILike)
  • Implemented environment variable fallbacks for provider base URLs
  • Added clear UI feedback for URL configuration and model management
  • Enhanced user guidance with configuration help sections

Key Improvements

  1. URL Configuration

    • Now respects environment variables from .env.local
    • Provides fallback defaults if no URLs are configured
    • Added visual feedback when editing URLs
    • Settings persist across page reloads
  2. Ollama Integration

    • Added restart server notifications after model changes
    • Improved error handling and user feedback
    • Clear instructions for model management
    • Dynamic base URL support for API calls
  3. UI Enhancements

    • Added configuration help section with clear instructions
    • Visual indicators for URL editing state
    • Better toast messages for model operations
    • Improved error state handling

Technical Details

  • Removed hardcoded URLs in favor of environment-based configuration
  • Added proper type safety for provider configurations
  • Improved error handling and user feedback
  • Settings persistence through the application's settings store

Testing Notes

  • Verify URL configuration persists after page reload
  • Test model installation and deletion notifications
  • Confirm environment variable overrides work as expected
  • Check error handling when services are unavailable

@Stijnus Stijnus changed the title fix: Update LocalProvidersTab.tsx fix: update LocalProvidersTab.tsx Feb 22, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant