Skip to content

Conversation

@brunobergher
Copy link
Collaborator

@brunobergher brunobergher commented Dec 23, 2025

Considering adoption and how much easier it is to get started with the Roo provider, this makes it the default path, while still leaving other providers as a secondary option. Plus other UX niceties.

Screenshot 2025-12-23 at 17 47 03
Screenshot 2025-12-23 at 17 47 08
Screenshot 2025-12-23 at 17 47 23

Important

Enhances signup UX by defaulting to Roo provider, adds UI improvements, and updates translations.

  • Behavior:
    • Default signup path set to Roo provider, with other providers as secondary options.
    • Added flag roo-auth-skip-model in WebAuthService.ts to handle cases where no model is selected during signup.
  • UI Enhancements:
    • Updated WelcomeViewProvider.tsx to include a landing screen and provider selection screen.
    • Added animations and UI elements in RooHero.tsx.
    • Adjusted ApiOptions.tsx to filter out Roo provider on the welcome screen.
  • Testing:
    • Added tests in WelcomeViewProvider.spec.tsx to cover new signup flow and UI changes.
  • Translations:
    • Updated translation files for multiple languages to reflect new UI text and options.
  • Styles:
    • Added new animations ground-slide and sun in index.css.

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

@roomote
Copy link
Contributor

roomote bot commented Dec 23, 2025

Rooviewer Clock   See task on Roo Cloud

All issues have been resolved. No new issues found.

  • Missing TriangleAlert mock in lucide-react test mock
  • Tests reference removed translation key welcome:providerSignup.rooCloudProvider (should be rooCloudDescription) - all instances updated to welcome:greeting
  • Missing Brain icon mock in lucide-react test mock - component imports and uses Brain but mock doesn't include it
  • Incorrect translation key test IDs in provider selection test - uses trans-welcome:introduction and trans-welcome:chooseProvider but should use trans-welcome:providerSignup.chooseProvider
Previous reviews

Mention @roomote in a comment to request specific changes to this pull request or fix all unresolved issues.

brunobergher and others added 2 commits December 23, 2025 12:30
…r.spec.tsx

Co-authored-by: roomote[bot] <219738659+roomote[bot]@users.noreply.github.com>
…r.spec.tsx

Co-authored-by: roomote[bot] <219738659+roomote[bot]@users.noreply.github.com>
@hannesrudolph hannesrudolph added the Issue/PR - Triage New issue. Needs quick review to confirm validity and assign labels. label Dec 23, 2025
- Add missing TriangleAlert icon to lucide-react mock
- Update test assertions to match new component structure
- Replace references to 'rooCloudProvider' heading with 'greeting'
- Update 'Use Roo Code Provider' to 'backToRoo' translation key
- All 9 tests now passing
brunobergher and others added 4 commits December 23, 2025 18:05
…r.spec.tsx

Co-authored-by: roomote[bot] <219738659+roomote[bot]@users.noreply.github.com>
…r.spec.tsx

Co-authored-by: roomote[bot] <219738659+roomote[bot]@users.noreply.github.com>
@brunobergher brunobergher marked this pull request as ready for review December 23, 2025 18:42
@dosubot dosubot bot added size:XL This PR changes 500-999 lines, ignoring generated files. UI/UX UI/UX related or focused labels Dec 23, 2025
@roomote
Copy link
Contributor

roomote bot commented Dec 23, 2025

Rooviewer Clock   See task on Roo Cloud

All issues have been resolved. Tests pass.

  • Missing TriangleAlert mock in lucide-react test mock
  • Tests reference removed translation key welcome:providerSignup.rooCloudProvider (should be rooCloudDescription) - all instances updated to welcome:greeting
  • Missing Brain icon mock in lucide-react test mock - component imports and uses Brain but mock doesn't include it
  • Incorrect translation key test IDs in provider selection test - uses trans-welcome:introduction and trans-welcome:chooseProvider but should use trans-welcome:providerSignup.chooseProvider
Previous reviews

Mention @roomote in a comment to request specific changes to this pull request or fix all unresolved issues.

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

Labels

Issue/PR - Triage New issue. Needs quick review to confirm validity and assign labels. lgtm This PR has been approved by a maintainer size:XL This PR changes 500-999 lines, ignoring generated files. UI/UX UI/UX related or focused

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

4 participants