Update Join page with Discord invite link#16
Conversation
|
Someone is attempting to deploy a commit to the bitsnbytes' projects Team on Vercel. A member of the Team first needs to authorize it. |
📝 WalkthroughWalkthroughThe join page migration replaces Notion form links with a Discord invite URL, updating the constant definition and both primary CTA buttons to point to Discord with refreshed labels and text throughout the page. ChangesDiscord CTA Links
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~3 minutes Poem
🚥 Pre-merge checks | ✅ 4 | ❌ 1❌ Failed checks (1 warning)
✅ Passed checks (4 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ 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 |
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 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 `@app/join/page.tsx`:
- Around line 144-150: The nearby informational text that still references
"weekly application review" and "7-day response" must be updated to match the
Discord CTA; locate the block near the Join Discord anchor
(href={DISCORD_INVITE_URL}, ArrowRight) in page.tsx and replace the stale copy
with a short, accurate message such as: "Join our Discord to chat with the
community and get help and updates in real time." Ensure the new sentence
replaces any mention of the application review schedule and retains existing
accessibility attributes and styling around the CTA.
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
| href={DISCORD_INVITE_URL} | ||
| target="_blank" | ||
| rel="noopener noreferrer" | ||
| aria-label="Open Bits&Bytes join form in Notion" | ||
| aria-label="Join Bits&Bytes Discord server" | ||
| > | ||
| Apply to Join | ||
| Join Discord | ||
| <ArrowRight className="h-6 w-6 shrink-0 transition-transform group-hover:translate-x-1" /> |
There was a problem hiding this comment.
Update stale “application review” copy near the Discord CTA.
After switching this CTA to Discord, the nearby message about weekly application review and 7-day response is now misleading.
✏️ Proposed copy fix
- We review applications weekly · You'll hear back within 7
- days
+ Join instantly on Discord · Introduce yourself to get started🤖 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 `@app/join/page.tsx` around lines 144 - 150, The nearby informational text that
still references "weekly application review" and "7-day response" must be
updated to match the Discord CTA; locate the block near the Join Discord anchor
(href={DISCORD_INVITE_URL}, ArrowRight) in page.tsx and replace the stale copy
with a short, accurate message such as: "Join our Discord to chat with the
community and get help and updates in real time." Ensure the new sentence
replaces any mention of the application review schedule and retains existing
accessibility attributes and styling around the CTA.
Summary by CodeRabbit