Skip to content

Conversation

@ComputelessComputer
Copy link
Collaborator

No description provided.

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Nov 28, 2025

📝 Walkthrough

Walkthrough

Adds a "Templates" navigation link to both the footer's Resources section and the header's features list, enabling consistent navigation access to the templates page across the site.

Changes

Cohort / File(s) Summary
Navigation link additions
apps/web/src/components/header.tsx, apps/web/src/components/footer.tsx
Adds new "Templates" navigation item to header features list and footer Resources section, each with corresponding route and styling.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

  • Both changes are straightforward single-line additions to existing navigation structures with no business logic or control-flow implications.

Possibly related PRs

Suggested reviewers

  • yujonglee

Pre-merge checks and finishing touches

❌ Failed checks (2 warnings)
Check name Status Explanation Resolution
Description check ⚠️ Warning No pull request description was provided by the author, which fails the basic requirement for documentation. Add a pull request description explaining the purpose of adding the Templates link and any relevant context or rationale for this change.
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 (1 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly and concisely describes the main change: adding a Templates link to both header and footer navigation components.
✨ 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 feat/add-templates-link

📜 Recent review details

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between b3f4524 and cb60ffd.

📒 Files selected for processing (2)
  • apps/web/src/components/footer.tsx (1 hunks)
  • apps/web/src/components/header.tsx (1 hunks)
🧰 Additional context used
📓 Path-based instructions (1)
**/*.{ts,tsx}

📄 CodeRabbit inference engine (AGENTS.md)

**/*.{ts,tsx}: Avoid creating a bunch of types/interfaces if they are not shared. Especially for function props, just inline them instead.
Never do manual state management for form/mutation. Use useForm (from tanstack-form) and useQuery/useMutation (from tanstack-query) instead for 99% of cases. Avoid patterns like setError.
If there are many classNames with conditional logic, use cn (import from @hypr/utils). It is similar to clsx. Always pass an array and split by logical grouping.
Use motion/react instead of framer-motion.

Files:

  • apps/web/src/components/header.tsx
  • apps/web/src/components/footer.tsx
⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (5)
  • GitHub Check: Redirect rules - hyprnote
  • GitHub Check: Header rules - hyprnote
  • GitHub Check: Pages changed - hyprnote
  • GitHub Check: fmt
  • GitHub Check: ci (macos, macos-14)
🔇 Additional comments (2)
apps/web/src/components/footer.tsx (1)

157-164: Footer Resources: Templates link is consistent and well‑placed

The new Templates link matches existing footer link patterns (structure, classes, and router Link usage) and aligns with the PR goal of surfacing templates; no issues found.

apps/web/src/components/header.tsx (1)

20-26: Header features: Templates item integrates cleanly

Adding Templates to featuresList follows the existing data-driven pattern and correctly propagates to both desktop and mobile menus, keeping navigation consistent with the footer.


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

@netlify
Copy link

netlify bot commented Nov 28, 2025

Deploy Preview for hyprnote ready!

Name Link
🔨 Latest commit cb60ffd
🔍 Latest deploy log https://app.netlify.com/projects/hyprnote/deploys/6928fec78274ff0008f50ab2
😎 Deploy Preview https://deploy-preview-1960--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.

@netlify
Copy link

netlify bot commented Nov 28, 2025

Deploy Preview for hyprnote-storybook ready!

Name Link
🔨 Latest commit cb60ffd
🔍 Latest deploy log https://app.netlify.com/projects/hyprnote-storybook/deploys/6928fec7de41140008c54fba
😎 Deploy Preview https://deploy-preview-1960--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.

@argos-ci
Copy link

argos-ci bot commented Nov 28, 2025

The latest updates on your projects. Learn more about Argos notifications ↗︎

Build Status Details Updated (UTC)
web (Inspect) ⚠️ Changes detected (Review) 3 changed Nov 28, 2025, 1:49 AM

@ComputelessComputer ComputelessComputer merged commit 917a349 into main Nov 28, 2025
11 of 12 checks passed
@ComputelessComputer ComputelessComputer deleted the feat/add-templates-link branch November 28, 2025 10:03
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