feat/fix(onboarding): completeness indicator + copy & close polish (#2-#6)#373
Merged
mcull merged 2 commits intoJul 1, 2026
Merged
Conversation
- Step 1 header now reinforces the fast-track ('name + a few basics') rather
than 'set up how you'll appear'; subtitle is step-aware.
- Just-in-time complete-profile prompts (lend/borrow/create) name the full
ask (photo AND verified address) so there's no surprise second step.
- Wizard gains a prominent Close affordance that exits back to the app when
it was opened from a just-in-time prompt (no longer trapped, Back only
walked wizard steps).
- Photo-rejection message is warm and on-brand instead of a raw model reason
(raw reason still logged for debugging).
Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
…cator (#2) The linear stepper made minimal-first entry feel like a required 3-step process. Replace it with a completeness indicator that shows the parts of the library card (Name & agreements / Photo / Address) and checks each off as it fills in — so photo/address read as 'add anytime,' not 'mandatory next.' - Pure tested profileCardStatus()/completedCardCount() helpers. - ProfileCompleteness component (done = check, current = filled ring, else hollow), plus a 'you can fill in the rest anytime' caption. - Removed the numbered Stepper + custom step-icon styling. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
mcull
changed the title
This was referenced Jul 1, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Onboarding UX pass from Marc's testing session. Stacked on #372 (shares
ProfileWizard.tsx) — retarget tomainonce #372 merges.#2 — Card-completeness indicator (replaces the 1-2-3 stepper). The linear numbered stepper made minimal-first entry feel like a required 3-step process. Replaced with a completeness indicator that shows the parts of the library card — Name & agreements / Photo / Address — and checks each off as it fills in, so photo/address read as "add anytime," not "mandatory next." Pure tested
profileCardStatus()helper; newProfileCompletenesscomponent; numbered Stepper + custom step-icon styling removed. Caption: "You can fill in the rest anytime."#3 — Step 1 copy. Card subtitle is now step-aware: Step 1 reinforces the fast track; later steps talk about photo/address.
#4 — JIT prompt names the full ask. Lend/borrow/create prompts now say a photo and verified address are needed (CTA "Finish profile"), instead of photo-only.
#5 — Close affordance. When opened from a just-in-time prompt (
?continue=1), the wizard shows a prominent Close (X) that exits back to the app (returnTo or/stacks) instead of trapping the user (Back only walked wizard steps).#6 — Friendly photo rejection. Warm, on-brand rejection note; the raw model reason is logged for debugging rather than shown.
Verification
tsc, eslint, unit tests (incl. newprofileCardStatus+profileSubmitBlockMessage), andnpm run buildall pass.For preview review
The completeness indicator is a first pass — worth eyeballing on the preview for visual weight vs. the old stepper, and whether we want to lean further into the physical "library card" metaphor.
🤖 Generated with Claude Code