Skip to content

fix: Onboarding / assistant switcher improvements #6573

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

Open
wants to merge 6 commits into
base: main
Choose a base branch
from

Conversation

sestinj
Copy link
Contributor

@sestinj sestinj commented Jul 11, 2025

Description

A number of small improvements to the onboarding flow + assistant switcher

Checklist

  • I've read the contributing guide
  • The relevant docs, if any, have been updated or created
  • The relevant tests, if any, have been updated or created

Tests

These are small, safe UI changes


Summary by cubic

Improved the onboarding flow and assistant switcher by making UI updates and fixing confusing actions.

  • UI Improvements
    • Show a message when no assistants are found.
    • Always display the user’s email in the switcher.
    • Update the reload button to refresh assistants instead of creating a new one.
    • Remove blocking prompts so users are not forced to create an assistant.

@sestinj sestinj requested a review from a team as a code owner July 11, 2025 07:29
@sestinj sestinj requested review from tomasz-stefaniak and removed request for a team July 11, 2025 07:29
@dosubot dosubot bot added the size:M This PR changes 30-99 lines, ignoring generated files. label Jul 11, 2025
Copy link

netlify bot commented Jul 11, 2025

Deploy Preview for continuedev failed. Why did it fail? →

Name Link
🔨 Latest commit da0335a
🔍 Latest deploy log https://app.netlify.com/projects/continuedev/deploys/6870c17e6b80a000084dafcd

Copy link

⚠️ Conventional Commit Format

Your commit messages don't follow the conventional commit format, but this won't block your PR from being merged. We recommend downloading this extension if you are using VS Code.

Expected Format:

<type>[optional scope]: <description>

[optional body]

[optional footer(s)]

Examples:

  • feat: add changelog generation support
  • fix: resolve login redirect issue
  • docs: update README with new instructions
  • chore: update dependencies

Valid Types:

feat, fix, docs, style, refactor, perf, test, build, ci, chore, revert

This helps with:

  • 📝 Automatic changelog generation
  • 🚀 Automated semantic versioning
  • 📊 Better project history tracking

This is a non-blocking warning - your PR can still be merged without fixing this.

Copy link
Contributor

@cubic-dev-ai cubic-dev-ai bot left a comment

Choose a reason for hiding this comment

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

cubic found 1 issue across 3 files. Review it in cubic.dev

React with 👍 or 👎 to teach cubic. Tag @cubic-dev-ai to give specific feedback.

No assistants found
</div>
) : (
profiles?.map((profile, idx) => (
Copy link
Contributor

Choose a reason for hiding this comment

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

Using the array index as the key in a list can cause rendering issues if the list order changes. It's better to use a unique identifier such as profile.id as the key.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
size:M This PR changes 30-99 lines, ignoring generated files.
Projects
Status: Todo
Development

Successfully merging this pull request may close these issues.

1 participant