[navbar] Route Reach Us dropdown to GitHub - #18
Conversation
Replace mailto links with GitHub Issues/Discussions URLs for better issue tracking and feature request workflow. Email address remains visible in the footer but links to GitHub Issues. Update tests accordingly. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
ℹ️ Recent review info⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Free Run ID: 📒 Files selected for processing (2)
📝 WalkthroughWalkthroughThe ReachDevelopers component transitions from email-based contact links to GitHub-based issue and discussion links. The dropdown icon changes from Mail to GitBranch, menu items update with new labels, and all contact links redirect to GitHub URLs with appropriate parameters. Tests are updated to match the new UI and link expectations. Changes
Estimated code review effort🎯 2 (Simple) | ⏱️ ~10 minutes Poem
Note 🎁 Summarized by CodeRabbit FreeYour organization is on the Free plan. CodeRabbit will generate a high-level summary and a walkthrough for each pull request. For a comprehensive line-by-line review, please upgrade your subscription to CodeRabbit Pro by visiting https://app.coderabbit.ai/login. Comment |
- Parse --allowed-origins CLI flag (and FAILPROOFAI_ALLOWED_DEV_ORIGINS env var) to configure Next.js allowedDevOrigins for cross-origin dev access - Refactor reach-developers dropdown to use a fixed backdrop div for click-outside detection instead of a document mousedown listener - Fix reach-developers email link to use mailto: href - Make bin/failproofai.mjs executable - Add docs for --allowed-origins flag and non-localhost dashboard access Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Summary
mailto:links in the Reach Us dropdown with GitHub URLs:issues/new?labels=enhancementissues/new?labels=bugdiscussions/new?category=q-aMailicon forGitBranch(noGithubicon in the installed lucide-react version)mailto:target="_blank")Test plan
npm test— all 709 tests pass🤖 Generated with Claude Code