Add Volunteer nav link to employment platform - #202
Conversation
Links the primary nav to the Civic Tech DC employment platform (a separate Next.js/Supabase app deployed to platform.civictechdc.org). The two sites stay independently deployed; the only contract is the URL. Placed after Projects as an outbound absolute link, so it skips the internal aria-current logic the other nav items use. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
![gemini-code-assist[bot]](https://avatars.githubusercontent.com/in/956858?s=60&v=4){kind=small}
There was a problem hiding this comment.
Code Review
This pull request adds a 'Volunteer' navigation link to the header pointing to an external platform. The feedback suggests avoiding hardcoding this external URL directly in the template to facilitate local or staging environment testing, recommending instead to define it in _config.yml with a fallback.
Important
The consumer version of Gemini Code Assist on GitHub is being sunset. Starting June 18, 2026, new organization installations will be blocked, and all code review activity will officially cease on July 17, 2026.
For more details on the timeline and next steps, please review the Help Documentation.
| <li class="usa-nav__primary-item"> | ||
| <a class="usa-nav__link" href="https://platform.civictechdc.org"> | ||
| <span>Volunteer</span> | ||
| </a> | ||
| </li> |
There was a problem hiding this comment.
Hardcoding the external platform URL directly in the template makes it difficult to point to local or staging environments (such as http://localhost:3000 or a Vercel preview URL) during development and testing.
Consider defining the platform URL in _config.yml (e.g., platform_url: https://platform.civictechdc.org) and referencing it here with a fallback to the production URL.
<li class="usa-nav__primary-item">
<a class="usa-nav__link" href="{{ site.platform_url | default: 'https://platform.civictechdc.org' }}">
<span>Volunteer</span>
</a>
</li>
1 participant
What
Adds a Volunteer item to the primary navigation, linking to the Civic Tech DC employment platform at
https://platform.civictechdc.org.Why
The employment-platform is a separate Next.js + Supabase app (deployed to Vercel) that lets volunteers browse projects and find where to contribute. Rather than fold a database-backed app into this static Jekyll/GitHub Pages site, the two stay independently deployed and the only contract between them is the URL.
Changes
usa-nav__primary-itemin_includes/core/header.html, placed after Projects.aria-currentpage-matching logic the internal nav items use.Notes / out of scope
CNAMEforplatform.civictechdc.orgat Vercel and add the domain to the Vercel project. The link 404s until that DNS is live — merge once the platform is deployed (or accept a temporary dead link).npm run lintpasses.🤖 Generated with Claude Code