Skip to content

Polish waitlist submitted state - #292

Open
Archimeyuan wants to merge 1 commit into
bountydotnew:mainfrom
Archimeyuan:polish-waitlist-submitted-state
Open

Polish waitlist submitted state#292
Archimeyuan wants to merge 1 commit into
bountydotnew:mainfrom
Archimeyuan:polish-waitlist-submitted-state

Conversation

@Archimeyuan

@Archimeyuan Archimeyuan commented May 18, 2026

Copy link
Copy Markdown

BOUNTY.NEW

Closes #231
@bountydotnew submit

Summary

  • redesigned the submitted waitlist state as a compact confirmation card
  • replaced the misleading Position #0 fallback with status/community tiles
  • kept the change UI-only and preserved the existing waitlist submission flow

Validation

  • git diff --check -- apps/web/src/components/landing/waitlist-demo.tsx
  • npx bun run check:web (fails in existing @bounty/ui typecheck errors for unresolved @/... imports outside this change)

Summary by CodeRabbit

  • New Features

    • Redesigned waitlist success screen with updated status and community information cards.
    • Enhanced messaging to prompt users to check their inbox for next steps.
  • Style

    • Updated success card styling with new gradient divider and adjusted icon sizing.
    • Improved display of waitlist count with dynamic text option.

Review Change Stack

@bountydotnew

bountydotnew Bot commented May 18, 2026

Copy link
Copy Markdown

@Archimeyuan bounty.new is in early access. Link your GitHub account at https://bounty.new/integrations to get started.

@vercel

vercel Bot commented May 18, 2026

Copy link
Copy Markdown
Contributor

@Archimeyuan is attempting to deploy a commit to the Bounty Team on Vercel.

A member of the Team first needs to authorize it.

@coderabbitai

coderabbitai Bot commented May 18, 2026

Copy link
Copy Markdown
Contributor

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: b0ff3fc5-4ace-49ff-a00a-a5844fdf8ac6

📥 Commits

Reviewing files that changed from the base of the PR and between 0377c8f and 764cb3a.

📒 Files selected for processing (1)
  • apps/web/src/components/landing/waitlist-demo.tsx

📝 Walkthrough

Walkthrough

The waitlist success state UI is redesigned with updated card styling, new stat cards showing status and community metrics, and conditional rendering logic. The component now computes a hasWaitlistCount flag to determine whether to display the numeric waitlist count or the text "Growing" in the community stat.

Changes

Waitlist Success UI Redesign

Layer / File(s) Summary
Waitlist count state computation
apps/web/src/components/landing/waitlist-demo.tsx
Adds hasWaitlistCount boolean derived from waitlistCount > 0 to control conditional rendering in the success message.
Success card styling and structure
apps/web/src/components/landing/waitlist-demo.tsx
Updates success card container markup with a new gradient top divider and adjusts icon wrapper sizing to be responsive to compact mode.
Success message content and stat cards
apps/web/src/components/landing/waitlist-demo.tsx
Replaces the old position/reach-out message with new headline and subheadline text, introduces "Status" and "Community" stat cards, conditionally renders the community count using NumberFlow or 'Growing', and updates the "Next step" instruction.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Poem

🐰 A waitlist page gets fresh new clothes,
With glowing cards and gradient flows,
Count the joiners—or say "Growing" when few,
The success state shines in vibrant hue! ✨

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Docstring Coverage ⚠️ Warning Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%. Write docstrings for the functions missing them to satisfy the coverage threshold.
✅ Passed checks (4 passed)
Check name Status Explanation
Title check ✅ Passed The title 'Polish waitlist submitted state' directly relates to the primary change—redesigning the waitlist success UI—and concisely conveys the main objective.
Description check ✅ Passed The description includes required compliance notice, linked issue reference, clear summary of changes, and validation steps, meeting template requirements.
Linked Issues check ✅ Passed The changes fully address issue #231 by redesigning the waitlist submitted state UI, replacing the misleading 'Position #0' display with status/community tiles in a compact confirmation card.
Out of Scope Changes check ✅ Passed All changes are scoped to redesigning the waitlist-demo.tsx component's submitted state UI; no unrelated modifications detected beyond the stated objectives.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

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

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.

❤️ Share

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

@Archimeyuan

Copy link
Copy Markdown
Author

Thanks! I opened this PR for #231, but the integrations page shows "Early Access Required" for my account. I've joined the waitlist. Is there another way to link my GitHub account or register this PR as a submission?

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.

improve design of waitlist submitted state on landing page

1 participant