Skip to content

fix(onboarding): clean up referral step UI and error display#457

Merged
graycyrus merged 2 commits into
tinyhumansai:mainfrom
graycyrus:fix/onboarding-ui-cleanup
Apr 9, 2026
Merged

fix(onboarding): clean up referral step UI and error display#457
graycyrus merged 2 commits into
tinyhumansai:mainfrom
graycyrus:fix/onboarding-ui-cleanup

Conversation

@graycyrus
Copy link
Copy Markdown
Contributor

@graycyrus graycyrus commented Apr 9, 2026

Summary

  • Remove progress bar (step dots) from onboarding overlay
  • Remove back button from referral code step
  • Place "Skip for now" and "Apply code" as inline side-by-side buttons
  • Parse API error responses into user-friendly messages instead of showing raw JSON/HTTP details

Test plan

  • Verify onboarding overlay no longer shows progress dots at the top
  • Verify referral step shows "Skip for now" and "Apply code" buttons inline
  • Verify entering an invalid referral code shows a clean error like "Invalid referral code" instead of raw API response
  • Verify applying a valid referral code still works and advances to next step

Summary by CodeRabbit

Release Notes

  • New Features

    • Added "Skip for now" button in the referral step, allowing users to proceed with onboarding without applying a referral code.
    • Improved button layout and styling for better usability.
  • Refactor

    • Removed step progress indicator from the onboarding experience for a cleaner interface.
    • Enhanced error handling to display clearer messages during referral code validation.

Remove progress bar from onboarding overlay, remove back button from
referral step, place Skip/Apply buttons inline, and parse API errors
into user-friendly messages instead of showing raw response bodies.
@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai Bot commented Apr 9, 2026

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: ef6aabe4-7dbb-4a08-b241-eea8b9c2d732

📥 Commits

Reviewing files that changed from the base of the PR and between 5dbf9d4 and a45c945.

📒 Files selected for processing (2)
  • app/src/pages/onboarding/Onboarding.tsx
  • app/src/pages/onboarding/steps/ReferralApplyStep.tsx

📝 Walkthrough

Walkthrough

The pull request removes the progress indicator UI from the onboarding flow and refactors the referral step to make the onBack callback optional, improve error handling with more robust parsing logic, and reorganize action buttons into a two-button row layout.

Changes

Cohort / File(s) Summary
Progress Indicator Removal
app/src/pages/onboarding/Onboarding.tsx
Removed ProgressIndicator component import and rendering, eliminating computed pagination values (totalSteps, progressCurrentStep) that adjusted step display based on skipReferralStep flag.
Referral Step Refactoring
app/src/pages/onboarding/steps/ReferralApplyStep.tsx
Made onBack prop optional, enhanced error handling in handleApply with more flexible error parsing (checks error and message fields, attempts JSON extraction), and replaced button layout with a two-button row ("Skip for now" and "Apply code") removing the back button.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~12 minutes

Possibly related PRs

Poem

🐰 Progress fades like morning dew,
Buttons dance in rows anew,
Error parsing stands more tall,
Onboarding flows, refined and small! ✨

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

Comment @coderabbitai help to get the list of available commands and usage tips.

@graycyrus graycyrus merged commit 4c90de4 into tinyhumansai:main Apr 9, 2026
6 of 9 checks passed
AusAgentSmith pushed a commit to AusAgentSmith/openhuman that referenced this pull request May 23, 2026
…ansai#457)

* fix(onboarding): clean up referral step UI and error display

Remove progress bar from onboarding overlay, remove back button from
referral step, place Skip/Apply buttons inline, and parse API errors
into user-friendly messages instead of showing raw response bodies.

* fix: remove unused onBack prop from ReferralApplyStep destructuring
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.

1 participant