Fix broken home-site links and add internal careers page#31
Merged
GTC6244 merged 1 commit intoMay 27, 2026
Merged
Conversation
Repoint Community to developer.litprotocol.com (old /support/intro is 404), Governance to the Discord (discourse forum is decommissioned). Replace the disabled jobs.litprotocol.com link with an internal /careers page noting no open positions, and open it in the same tab from Footer, Header, and both nav menus.
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
There was a problem hiding this comment.
Pull request overview
This PR fixes broken outbound links used across the site header/footer/nav and replaces the deprecated external careers/jobs link with a new internal /careers page under the Next.js App Router.
Changes:
- Updated
COMMUNITY_LINKandGOVERNANCE_LINKdestinations to working URLs. - Repointed
CAREERS_LINKto an internal/careersroute and adjusted nav/footer behavior to open in the same tab. - Added a new
src/app/careers/page.tsxthat renders a simple “no openings” message inside the standardLayout.
Reviewed changes
Copilot reviewed 6 out of 6 changed files in this pull request and generated 2 comments.
Show a summary per file
| File | Description |
|---|---|
src/utils/constants.ts |
Updates link constants for Community, Governance, and Careers. |
src/components/NavMenu/NavMenu2.tsx |
Marks Careers as an internal (same-tab) link in the mobile nav menu model. |
src/components/NavMenu/NavMenu.tsx |
Removes _blank behavior for Careers in the legacy nav menu. |
src/components/Header/Header.tsx |
Marks Careers as internal in the header menu dropdown model. |
src/components/Footer/Footer.tsx |
Removes _blank behavior for Careers in the footer. |
src/app/careers/page.tsx |
Adds the internal Careers page content under /careers. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| export const AUDIT_LINK = | ||
| 'https://drive.google.com/drive/folders/1Rrht88iUkzpofwl1CvP9gEjqY60BKyFn'; | ||
| export const COMMUNITY_LINK = 'https://developer.litprotocol.com/support/intro'; | ||
| export const COMMUNITY_LINK = 'https://developer.litprotocol.com/'; |
| export const COMMUNITY_CONSTANT_LINK = 'https://discord.gg/yEJSBaznBX'; | ||
| export const COMPANY_CONSTANT_LINK = 'https://litprotocol.com'; | ||
| export const GOVERNANCE_LINK = 'https://litprotocol.discourse.group'; | ||
| export const GOVERNANCE_LINK = 'https://getlit.dev/chat'; |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Audited every link on the home site and fixed the three that were broken:
COMMUNITY_LINK(Header/NavMenu "Resources") →https://developer.litprotocol.com/support/intro404s; repointed tohttps://developer.litprotocol.com/.GOVERNANCE_LINK(Footer "Governance") →litprotocol.discourse.groupno longer resolves; repointed to the Discord (getlit.dev/chat).CAREERS_LINK(Footer/Header/NavMenu) →jobs.litprotocol.compage loads but the account is disabled; replaced with an internal/careerspage noting no current openings, and droppedtarget="_blank"so it opens in the same tab.Test plan
/and click Community → Resources, Footer → Governance, Footer/Header/Nav → Careers; verify destinations and same-tab behavior for Careers/careersdirectly and confirm "no open positions" copy + return-home link render inside the standard Layout