Skip to content

Add Thank You step to onboarding flow#70

Closed
tolbrino wants to merge 1 commit intomainfrom
claude/show-github-issues-tcnu7
Closed

Add Thank You step to onboarding flow#70
tolbrino wants to merge 1 commit intomainfrom
claude/show-github-issues-tcnu7

Conversation

@tolbrino
Copy link

Summary

Added a new final step (step 100) to the onboarding flow that displays a thank you message to users after they complete the feedback form. This provides better closure to the onboarding experience.

Key Changes

  • Created new ThankYou component (src/views/onboarding/100_ThankYou.tsx) that displays a thank you message with optional close button
  • Updated onboarding index to register the new step in STEP_NAMES and STEP_COMPONENTS mappings
  • Modified Feedback component to navigate to step 100 instead of immediately resetting the store when feedback is submitted
  • Changed Feedback to use setOnboardingStep instead of resetStore for better control over the flow

Implementation Details

  • The ThankYou component accepts optional className and lastEntry props to control styling and button visibility
  • The close button only appears when lastEntry is true, allowing flexibility in how the step is used
  • Feedback submission now transitions to the thank you page rather than immediately closing the onboarding flow
  • The thank you message thanks users for completing onboarding and providing feedback

https://claude.ai/code/session_01XZv2TCCJX7ifRBWgzroSGt

Show a "You made it to the end!" screen after users submit the feedback
form instead of immediately jumping back to the login screen. The screen
displays a thank you message and a Close button that resets the session.

https://claude.ai/code/session_01XZv2TCCJX7ifRBWgzroSGt
@thewanderingeditor
Copy link

Does this conflict with #68 ?

@esterlus
Copy link
Contributor

Yes this is an overly complex version of #68, closing.

Please reopen if current solution does not suffice

@esterlus esterlus closed this Mar 23, 2026
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.

4 participants