Skip to content

fix for active llm provider in settings#98

Merged
martyna-mindsdb merged 2 commits into
mainfrom
fix-settings-llm-provider-save
May 18, 2026
Merged

fix for active llm provider in settings#98
martyna-mindsdb merged 2 commits into
mainfrom
fix-settings-llm-provider-save

Conversation

@martyna-mindsdb
Copy link
Copy Markdown
Contributor

Description

when providing an API key during onboarding/startup, then the active provider is not visible as active in the Settings tab -- this PR fixes it

how to test:

  • remove local .anton dir
  • start new anton instance and provide an api key during onboarding
  • go to settings and check if there is one provider with a green circle next to it, indicating the active provider

Fixes #issue_number

Screenshots

Type of change

Please delete options that are not relevant.

  • 🐛 Bug fix (non-breaking change which fixes an issue)
  • ⚡ New feature (non-breaking change which adds functionality)
  • 🚨 Hotfix (non-breaking change which fixes an issue)
  • 📢 Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • 📄 This change requires a documentation update

Checklist:

  • My code follows the style guidelines of this project
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation
  • My changes generate no new warnings
  • I have checked my code and corrected any misspellings

Comment thread server/routes/settings.py Outdated
Copy link
Copy Markdown
Contributor

@tino097 tino097 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

one nitpick

@torrmal torrmal enabled auto-merge May 16, 2026 22:10
@torrmal torrmal disabled auto-merge May 16, 2026 22:10
@martyna-mindsdb martyna-mindsdb enabled auto-merge May 18, 2026 09:35
@martyna-mindsdb
Copy link
Copy Markdown
Contributor Author

@tino097
Can you please resolve the comment so I can merge it?

@martyna-mindsdb martyna-mindsdb added this pull request to the merge queue May 18, 2026
Merged via the queue into main with commit e74580f May 18, 2026
8 checks passed
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.

3 participants