prototype: Start a community wizard (ports the app's /start flow) - #4
prototype: Start a community wizard (ports the app's /start flow)#4abullaisi wants to merge 1 commit into
Conversation
3-step onboarding (Welcome > Brand > You are live) in prototype idiom: real stepper classes, pay-col centered column, K state gate + fake tx, k_creator_brand persistence. Wires the 09 FOR PARTNERS placeholder CTA on index.html to the new screen. SPEC.md gets the Screen 8 brief. Copy ported verbatim from packages/web /start. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
📝 WalkthroughWalkthroughAdds a wallet-gated three-step community onboarding page, connects the existing creator CTA to it, and implements community-name validation, local storage, activity logging, and live-state progression. ChangesCommunity onboarding
Estimated code review effort: 3 (Moderate) | ~20 minutes Sequence Diagram(s)sequenceDiagram
actor Creator
participant starthtml as start.html
participant K
participant BrowserStorage as localStorage
Creator->>starthtml: Start a community
starthtml->>K: Check connected wallet
Creator->>starthtml: Submit community name
starthtml->>BrowserStorage: Save community name
starthtml->>K: Log community creation activity
starthtml->>starthtml: Advance to Live card
Possibly related PRs
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 3
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Inline comments:
In `@prototype/start.html`:
- Line 86: Update the brand text in the paragraph containing the live-name span
to use the consistently capitalized “Komunify” form instead of “komunify”.
- Line 26: Update the logo anchor in the shared topbar markup to match the
SPEC.md page skeleton exactly, removing the inline img element and preserving
the consistent logo link structure used across screens.
- Around line 92-94: Update the footer containing the reset-proto link to
restore the standard text prefix specified by the shared page skeleton in
SPEC.md, while preserving the existing “Reset demo” link.
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: defaults
Review profile: CHILL
Plan: Pro Plus
Run ID: b06e6b62-9031-446b-a0b1-e9e0c6470a04
📒 Files selected for processing (3)
prototype/SPEC.mdprototype/index.htmlprototype/start.html
| <body> | ||
| <main class="shell"> | ||
| <div class="topbar"> | ||
| <a class="logo" href="index.html"><img src="logo-mark.png" alt="" class="logo-mark">Komunify</a> |
There was a problem hiding this comment.
📐 Maintainability & Code Quality | 🟡 Minor | ⚡ Quick win
Align logo markup with the shared page skeleton.
The logo link includes an inline <img...> tag, which deviates from the SPEC.md shared page skeleton that dictates the topbar markup must remain identical on every screen.
🛠️ Proposed fix
- <a class="logo" href="index.html"><img src="logo-mark.png" alt="" class="logo-mark">Komunify</a>
+ <a class="logo" href="index.html">Komunify</a>📝 Committable suggestion
‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.
| <a class="logo" href="index.html"><img src="logo-mark.png" alt="" class="logo-mark">Komunify</a> | |
| <a class="logo" href="index.html">Komunify</a> |
🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
In `@prototype/start.html` at line 26, Update the logo anchor in the shared topbar
markup to match the SPEC.md page skeleton exactly, removing the inline img
element and preserving the consistent logo link structure used across screens.
| <section class="card center hidden" id="live-card"> | ||
| <div class="num-label"><span class="num">12</span> LIVE</div> | ||
| <h2>You are live</h2> | ||
| <p><span id="live-name">Your community</span> is now on komunify.</p> |
There was a problem hiding this comment.
📐 Maintainability & Code Quality | 🟡 Minor | ⚡ Quick win
Capitalize the brand name.
Ensure the brand name is consistently capitalized.
📝 Proposed fix
- <p><span id="live-name">Your community</span> is now on komunify.</p>
+ <p><span id="live-name">Your community</span> is now on Komunify.</p>📝 Committable suggestion
‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.
| <p><span id="live-name">Your community</span> is now on komunify.</p> | |
| <p><span id="live-name">Your community</span> is now on Komunify.</p> |
🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
In `@prototype/start.html` at line 86, Update the brand text in the paragraph
containing the live-name span to use the consistently capitalized “Komunify”
form instead of “komunify”.
| <footer> | ||
| <a href="#" id="reset-proto">Reset demo</a> | ||
| </footer> |
There was a problem hiding this comment.
📐 Maintainability & Code Quality | 🟠 Major | ⚡ Quick win
Restore missing footer text to match the shared skeleton.
The footer is missing the standard text prefix defined in the SPEC.md shared page skeleton. This omission breaks the expected global visual design contract.
🛠️ Proposed fix
<footer>
+ <span class="label">Prototype</span> · Komunify on Stellar testnet ·
<a href="#" id="reset-proto">Reset demo</a>
</footer>📝 Committable suggestion
‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.
| <footer> | |
| <a href="#" id="reset-proto">Reset demo</a> | |
| </footer> | |
| <footer> | |
| <span class="label">Prototype</span> · Komunify on Stellar testnet · | |
| <a href="#" id="reset-proto">Reset demo</a> | |
| </footer> |
🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
In `@prototype/start.html` around lines 92 - 94, Update the footer containing the
reset-proto link to restore the standard text prefix specified by the shared
page skeleton in SPEC.md, while preserving the existing “Reset demo” link.
1 participant
Adds the missing creator-side flow to the clickable prototype: a 3-step "Start a community" wizard at prototype/start.html, ported from the real app's /start (same steps, same copy: Welcome > Brand > You are live).
🤖 Generated with Claude Code
Summary by CodeRabbit
New Features
Improvements
Documentation