Skip to content

Conversation

@cte
Copy link
Collaborator

@cte cte commented Mar 17, 2025

Context

Screenshot 2025-03-17 at 1 59 28 PM

Implementation

Screenshots

before after

How to Test

Get in Touch


Important

Add feature to select specific providers for OpenRouter with UI support and translations.

  • Behavior:
    • Add ability to choose specific provider for OpenRouter in openrouter.ts.
    • Default provider set to [default] if no specific provider is chosen.
  • UI Components:
    • Add provider selection dropdown in ApiOptions.tsx.
    • Use useOpenRouterModelProviders hook in ApiOptions.tsx for fetching providers.
    • Integrate QueryClientProvider in App.tsx and related test files.
  • Hooks:
    • New hook useOpenRouterModelProviders in useOpenRouterModelProviders.ts for fetching provider data.
  • Translations:
    • Add translations for provider routing in multiple language files including en, es, fr, hi, it, ja, ko, pl, pt-BR, tr, vi, zh-CN, zh-TW.

This description was created by Ellipsis for ff9792d. It will automatically update as commits are pushed.

@changeset-bot
Copy link

changeset-bot bot commented Mar 17, 2025

🦋 Changeset detected

Latest commit: ff9792d

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
roo-cline Minor

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@dosubot dosubot bot added size:L This PR changes 100-499 lines, ignoring generated files. enhancement New feature or request labels Mar 17, 2025
@hannesrudolph hannesrudolph moved this from New to PR [Greenlit] in Roo Code Roadmap Mar 17, 2025
@cte
Copy link
Collaborator Author

cte commented Mar 17, 2025

@PhunkyBob - I reworked this a bit to clean up the useEffect calls - hope you don't mind. Thanks for doing the heavy lifting on this!

@dosubot dosubot bot added the lgtm This PR has been approved by a maintainer label Mar 17, 2025
@cte cte force-pushed the cte/openrouter-specific-provider branch from fca535f to a4978b3 Compare March 17, 2025 22:19
@cte cte merged commit eb74f02 into main Mar 17, 2025
9 of 10 checks passed
@cte cte deleted the cte/openrouter-specific-provider branch March 17, 2025 22:40
@github-project-automation github-project-automation bot moved this from PR [Greenlit] to Done in Roo Code Roadmap Mar 17, 2025
@PhunkyBob
Copy link

@PhunkyBob - I reworked this a bit to clean up the useEffect calls - hope you don't mind. Thanks for doing the heavy lifting on this!

No problem!
I'm new to React / TypeScript (I am more a Python guy), so I don't know the best practice yet.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request lgtm This PR has been approved by a maintainer size:L This PR changes 100-499 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants