Skip to content

Update Join page with Discord invite link#16

Open
sparshsharmadev wants to merge 1 commit into
gobitsnbytes:mainfrom
sparshsharmadev:update-join-link-discord
Open

Update Join page with Discord invite link#16
sparshsharmadev wants to merge 1 commit into
gobitsnbytes:mainfrom
sparshsharmadev:update-join-link-discord

Conversation

@sparshsharmadev
Copy link
Copy Markdown

@sparshsharmadev sparshsharmadev commented Jun 1, 2026

Summary by CodeRabbit

  • Updates
    • Join page now directs users to Discord instead of an external form
    • Main action button updated with Discord invite link for community access
    • Footer call-to-action button synchronized with matching Discord invite
    • All onboarding pathways consolidated through single Discord entry point

@vercel
Copy link
Copy Markdown

vercel Bot commented Jun 1, 2026

Someone is attempting to deploy a commit to the bitsnbytes' projects Team on Vercel.

A member of the Team first needs to authorize it.

@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented Jun 1, 2026

Review Change Stack

📝 Walkthrough

Walkthrough

The 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.

Changes

Discord CTA Links

Layer / File(s) Summary
Replace Notion join form with Discord invite links
app/join/page.tsx
Define new DISCORD_INVITE_URL constant and update both main and bottom CTA buttons to target Discord invite with refreshed button text and accessibility labels.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Poem

A rabbit hops to Discord today, 🐰
Trading forms for a friendlier way,
Links now lead where the rabbits all play,
Invite and connect—hooray, hooray! 🎉
Join the warren, don't delay! 🐇

🚥 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
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The pull request title directly and accurately summarizes the main change: updating the join page to use Discord invite links instead of Notion form links.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.

✏️ 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.

Copy link
Copy Markdown

@coderabbitai coderabbitai Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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

ℹ️ Review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: 7a568495-1822-4e34-81ee-6ebe0068df16

📥 Commits

Reviewing files that changed from the base of the PR and between cfe373e and 1c0b54d.

📒 Files selected for processing (1)
  • app/join/page.tsx

Comment thread app/join/page.tsx
Comment on lines +144 to 150
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" />
Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

⚠️ Potential issue | 🟡 Minor | ⚡ Quick win

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&apos;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.

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