Skip to content

[codex] Add Discord links to site navigation#5

Open
andrewtdiz wants to merge 2 commits into
PerryTS:mainfrom
andrewtdiz:codex/add-discord-links
Open

[codex] Add Discord links to site navigation#5
andrewtdiz wants to merge 2 commits into
PerryTS:mainfrom
andrewtdiz:codex/add-discord-links

Conversation

@andrewtdiz

@andrewtdiz andrewtdiz commented Jun 13, 2026

Copy link
Copy Markdown

Summary

  • Add Perry's Discord invite to the desktop and mobile header navigation between Docs and GitHub.
  • Add Discord as the first link in the footer Community column.
  • Add a shared nav.discord label across all locale message files and tighten desktop nav spacing for medium widths.

Validation

  • Parsed all 13 src/messages/*.json files with Node.
  • Ran git diff --check.
  • Ran npm run build:site.
  • Verified the built static site in Browser at desktop, 900px medium, and mobile widths:
    • desktop header shows Discord in order with the direct invite URL
    • medium-width header stays on one row with no overlap/cramped language switcher
    • mobile menu includes Discord between Docs and GitHub
    • footer Community column starts with Discord

Summary by CodeRabbit

  • New Features
    • Discord link added to header and footer navigation with a dedicated icon and localized label.
    • Localization keys for the Discord navigation label added across supported languages: English, German, Spanish, French, Indonesian, Italian, Japanese, Korean, Portuguese, Thai, Turkish, Vietnamese, and Chinese.

@coderabbitai

coderabbitai Bot commented Jun 13, 2026

Copy link
Copy Markdown

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro Plus

Run ID: 9b3bec4c-1007-473a-8318-a933d0b4f21d

📥 Commits

Reviewing files that changed from the base of the PR and between b97c5b8 and 4d46760.

⛔ Files ignored due to path filters (1)
  • public/discord.svg is excluded by !**/*.svg
📒 Files selected for processing (1)
  • src/components/Header.tsx
🚧 Files skipped from review as they are similar to previous changes (1)
  • src/components/Header.tsx

📝 Walkthrough

Walkthrough

Adds a shared DISCORD_URL and Discord links to Header (desktop + mobile) and Footer, and adds nav.discord translations across language message files.

Changes

Discord Navigation Integration

Layer / File(s) Summary
Header Discord navigation with SVG icon
src/components/Header.tsx
Adds DISCORD_URL constant, adjusts desktop nav gap, and renders Discord links with /discord.svg and translated nav.discord in desktop and mobile navigation.
Footer Discord community link
src/components/Footer.tsx
Adds DISCORD_URL constant and renders a community Discord link that opens in a new tab with rel="noopener noreferrer", using the translated label.
Internationalization for Discord label
src/messages/en.json, src/messages/de.json, src/messages/es.json, src/messages/fr.json, src/messages/id.json, src/messages/it.json, src/messages/ja.json, src/messages/ko.json, src/messages/pt.json, src/messages/th.json, src/messages/tr.json, src/messages/vi.json, src/messages/zh-Hans.json
Adds nav.discord translation key with the label "Discord" across all listed locales.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~8 minutes

Poem

🐰 A tiny link hops into place,
Discord icon greets every space,
Header, footer, languages sing—
Invite the chat, let voices spring,
A rabbit cheers: "Join the trace!"

🚥 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 title '[codex] Add Discord links to site navigation' clearly and concisely summarizes the main change: adding Discord links to the site's navigation across header and footer.
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

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

@andrewtdiz andrewtdiz marked this pull request as ready for review June 13, 2026 16:06
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