Skip to content

fix: update error boundary support link to help center - #262

Merged
paulbalaji merged 2 commits into
mainfrom
fix/error-boundary-help-link
Jan 29, 2026
Merged

fix: update error boundary support link to help center#262
paulbalaji merged 2 commits into
mainfrom
fix/error-boundary-help-link

Conversation

@paulbalaji

@paulbalaji paulbalaji commented Jan 29, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • Add help URL constant to links.ts
  • Update ErrorBoundary to link to Hyperlane Help Center instead of Discord
  • Refactor Footer to use links.help and links.jobs constants instead of hardcoded URLs

Test plan

  • Trigger an error to verify the error boundary shows the correct link
  • Verify the footer Support link works correctly

🤖 Generated with Claude Code

Summary by CodeRabbit

  • Updates
    • Support link now directs users to the Hyperlane Help Center instead of Discord.
    • Footer navigation updated to use centralized link constants for the Help Center and Careers, and a new centralized Help URL was added.

✏️ Tip: You can customize this high-level summary in your review settings.

- Add help URL constant to links.ts
- Update ErrorBoundary to link to Help Center instead of Discord
- Refactor Footer to use links constants for Support and Careers

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
@paulbalaji
paulbalaji requested a review from Xaroz as a code owner January 29, 2026 11:18
@vercel

vercel Bot commented Jan 29, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Review Updated (UTC)
hyperlane-explorer Ready Ready Preview, Comment Jan 29, 2026 11:26am

Request Review

@coderabbitai

coderabbitai Bot commented Jan 29, 2026

Copy link
Copy Markdown
📝 Walkthrough

Walkthrough

Adds a new help URL to centralized link constants and replaces hard-coded Help Center and jobs URLs in the ErrorBoundary and Footer components to reference those constants.

Changes

Cohort / File(s) Summary
Link Constants
src/consts/links.ts
Added a new help property to the exported links object for the Help Center URL.
Component Link References
src/components/errors/ErrorBoundary.tsx, src/components/nav/Footer.tsx
Replaced hard-coded Help Center and jobs URLs with links.help and links.jobs constants; updated displayed support text in ErrorBoundary accordingly.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~8 minutes

Poem

A wee tweak to paths, tidy and bright,
One spot for links, keeps the code polite,
No more huntin' round for a stray URL,
Clean and cozy — that works for me well. 🐸✨

🚥 Pre-merge checks | ✅ 2 | ❌ 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 (2 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly summarizes the main change: updating the ErrorBoundary support link from Discord to the Help Center, which aligns with the primary objective of the PR.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing touches
  • 📝 Generate docstrings
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch fix/error-boundary-help-link

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

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

@coderabbitai coderabbitai Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Actionable comments posted: 1

🤖 Fix all issues with AI agents
In `@src/components/errors/ErrorBoundary.tsx`:
- Around line 11-12: Change the double-quoted JSX string literals to single
quotes and wrap the long <a> element line in ErrorBoundary.tsx so it stays under
100 characters; specifically update the JSX attributes on the <a
href={links.help} target="_blank" rel="noopener noreferrer" className="mt-5
text-sm"> element to use single quotes for href/target/rel/className and break
the tag/content across multiple lines (you can also break the inner <span
className="underline underline-offset-2"> text onto its own line) to keep each
line <100 chars while preserving the same content and props.

Comment thread src/components/errors/ErrorBoundary.tsx Outdated
@paulbalaji
paulbalaji merged commit 4c53bf4 into main Jan 29, 2026
9 checks passed
@paulbalaji
paulbalaji deleted the fix/error-boundary-help-link branch January 29, 2026 14:47
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.

2 participants