Skip to content

[navbar] Route Reach Us dropdown to GitHub - #18

Merged
NiveditJain merged 2 commits into
mainfrom
ef-19
Apr 7, 2026
Merged

[navbar] Route Reach Us dropdown to GitHub#18
NiveditJain merged 2 commits into
mainfrom
ef-19

Conversation

@NiveditJain

Copy link
Copy Markdown
Member

Summary

  • Replaced all mailto: links in the Reach Us dropdown with GitHub URLs:
    • Request a Featureissues/new?labels=enhancement
    • Report an Issueissues/new?labels=bug
    • Ask a Questiondiscussions/new?category=q-a
  • Swapped Mail icon for GitBranch (no Github icon in the installed lucide-react version)
  • Email address kept visible at the bottom of the dropdown, but now links to GitHub Issues instead of mailto:
  • All links open in a new tab (target="_blank")
  • Updated tests to assert GitHub URLs instead of mailto

Test plan

  • Open the Reach Us dropdown — verify 3 items with GitHub links
  • Click each item — confirm it opens the correct GitHub page in a new tab
  • Confirm email is still shown at bottom and links to GitHub Issues
  • Run npm test — all 709 tests pass

🤖 Generated with Claude Code

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>
@coderabbitai

coderabbitai Bot commented Apr 6, 2026

Copy link
Copy Markdown
ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Free

Run ID: f9fa372a-8cc6-4647-8cab-43f46358a654

📥 Commits

Reviewing files that changed from the base of the PR and between ca824cc and 53ba533.

📒 Files selected for processing (2)
  • __tests__/components/reach-developers.test.tsx
  • components/reach-developers.tsx

📝 Walkthrough

Walkthrough

The 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

Cohort / File(s) Summary
Component Implementation
components/reach-developers.tsx
Migrated contact mechanism from email (mailto:) to GitHub-based links. Icon updated from Mail to GitBranch. Menu items now point to GitHub Issues creation and Discussions Q&A category with target="_blank" and rel="noopener noreferrer". Footer link changed from direct email anchor to GitHub Issues URL.
Test Suite
__tests__/components/reach-developers.test.tsx
Updated test assertions to reflect new UI text ("Report an Issue", "Ask a Question"), new GitHub URL patterns with github.com/exospherehost/failproofai and query parameters, and adjusted close-behavior checks to target updated menu labels.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Poem

🐰 From inboxes we hop away,

To GitHub's halls, we'll always stay,

With branches green and issues bright,

Our developers reach new height! 📬➜🐙


Note

🎁 Summarized by CodeRabbit Free

Your 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 @coderabbitai help to get the list of available commands and usage tips.

- 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>
@NiveditJain
NiveditJain merged commit 1c12bae into main Apr 7, 2026
7 checks passed
@NiveditJain
NiveditJain deleted the ef-19 branch April 21, 2026 01:31
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