Skip to content

refactor(web): replace external links with internal routing using Link#2180

Merged
ComputelessComputer merged 3 commits intomainfrom
feat/replace-external-links-routing
Dec 9, 2025
Merged

refactor(web): replace external links with internal routing using Link#2180
ComputelessComputer merged 3 commits intomainfrom
feat/replace-external-links-routing

Conversation

@ComputelessComputer
Copy link
Collaborator

Overview

  • Replaced external links with internal routing using Next.js Link component
  • Improved navigation performance and user experience
  • Ensured consistent routing across the web application

Changes

  • Migrated external anchor tags to Next.js Link component
  • Optimized client-side navigation
  • Enhanced application routing strategy

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Dec 9, 2025

Warning

Rate limit exceeded

@ComputelessComputer has exceeded the limit for the number of commits or files that can be reviewed per hour. Please wait 24 minutes and 7 seconds before requesting another review.

⌛ How to resolve this issue?

After the wait time has elapsed, a review can be triggered using the @coderabbitai review command as a PR comment. Alternatively, push new commits to this PR.

We recommend that you space out your commits to avoid hitting the rate limit.

🚦 How do rate limits work?

CodeRabbit enforces hourly rate limits for each developer per organization.

Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout.

Please see our FAQ for further information.

📥 Commits

Reviewing files that changed from the base of the PR and between e2f21ab and 3649e52.

⛔ Files ignored due to path filters (1)
  • apps/web/src/routeTree.gen.ts is excluded by !**/*.gen.ts
📒 Files selected for processing (8)
  • apps/web/src/routes/_view/download/apple.tsx (1 hunks)
  • apps/web/src/routes/_view/integrations/$category.$slug.tsx (2 hunks)
  • apps/web/src/routes/_view/product/ai-assistant.tsx (3 hunks)
  • apps/web/src/routes/_view/product/ai-notetaking.tsx (3 hunks)
  • apps/web/src/routes/_view/product/local-ai.tsx (1 hunks)
  • apps/web/src/routes/_view/product/mini-apps.tsx (4 hunks)
  • apps/web/src/routes/_view/product/self-hosting.tsx (1 hunks)
  • apps/web/src/routes/_view/vs/$slug.tsx (2 hunks)
📝 Walkthrough

Walkthrough

This PR replaces anchor tags with <Link> components for internal navigation to the download page across three sections of the ai-notetaking page, converting from absolute URLs to relative routing without changing control flow or error handling.

Changes

Cohort / File(s) Summary
Navigation component replacement
apps/web/src/routes/_view/product/ai-notetaking.tsx
Replaced three anchor tag instances with <Link> components in HeroSection and CTASection; updated href attributes from external URLs to relative /download routes

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~5 minutes

  • Single file with repetitive, straightforward component replacements
  • No logic changes or functional modifications
  • Consistent pattern applied across three locations

Possibly related PRs

  • ui changes #1796: Also replaces anchor-based internal navigation with <Link> components in the same ai-notetaking.tsx file
  • landing #1649: Modifies the same file and touches HeroSection/CTASection components
  • c-branch-2 #1894: Makes overlapping UI/component changes in the same ai-notetaking.tsx file

Suggested reviewers

  • yujonglee

Pre-merge checks and finishing touches

❌ 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%. You can run @coderabbitai generate docstrings to improve docstring coverage.
✅ Passed checks (2 passed)
Check name Status Explanation
Title check ✅ Passed The title accurately and clearly describes the main change: replacing external links with Next.js Link components for internal routing.
Description check ✅ Passed The description is clearly related to the changeset, explaining the motivation (performance, UX, consistency) and the specific changes made (anchor to Link migration).

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.

@netlify
Copy link

netlify bot commented Dec 9, 2025

Deploy Preview for hyprnote-storybook ready!

Name Link
🔨 Latest commit 3649e52
🔍 Latest deploy log https://app.netlify.com/projects/hyprnote-storybook/deploys/69379ea5b6df35000861f8c7
😎 Deploy Preview https://deploy-preview-2180--hyprnote-storybook.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@netlify
Copy link

netlify bot commented Dec 9, 2025

Deploy Preview for hyprnote ready!

Name Link
🔨 Latest commit 3649e52
🔍 Latest deploy log https://app.netlify.com/projects/hyprnote/deploys/69379ea56f7fab0008e25e13
😎 Deploy Preview https://deploy-preview-2180--hyprnote.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@ComputelessComputer ComputelessComputer force-pushed the feat/replace-external-links-routing branch 2 times, most recently from 5d47d4c to a243d25 Compare December 9, 2025 03:59
@ComputelessComputer ComputelessComputer force-pushed the feat/replace-external-links-routing branch from a243d25 to 3649e52 Compare December 9, 2025 03:59
@ComputelessComputer ComputelessComputer merged commit 65bdb56 into main Dec 9, 2025
13 checks passed
@ComputelessComputer ComputelessComputer deleted the feat/replace-external-links-routing branch December 9, 2025 04:05
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