-
Notifications
You must be signed in to change notification settings - Fork 433
feat(web): update meta titles for templates, docs, blog, and changelog pages #2020
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
…g pages
- Templates: 'Meeting Templates - Hyprnote Templates'
- Docs: '{title} - Hyprnote Documentation'
- Blog: '{title} - Hyprnote Blog'
- Changelog: '{title} - Hyprnote Changelog'
Co-Authored-By: john@hyprnote.com <john@hyprnote.com>
🤖 Devin AI EngineerI'll be helping with this pull request! Here's what you should know: ✅ I will automatically:
Note: I can only respond to comments from users who have write access to this repository. ⚙️ Control Options:
|
✅ Deploy Preview for hyprnote-storybook ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
📝 WalkthroughWalkthroughThis PR updates SEO metadata titles across multiple route files (blog, changelog, docs, templates) by standardizing their format and appending appropriate section identifiers to improve search engine optimization and social media sharing. Changes
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~5 minutes All changes are cosmetic, metadata-only updates with consistent patterns across files. No logic modifications, control flow changes, or behavioral alterations detected. Possibly related PRs
Suggested reviewers
Pre-merge checks and finishing touches✅ Passed checks (3 passed)
✨ Finishing touches
🧪 Generate unit tests (beta)
Comment |
✅ Deploy Preview for hyprnote ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Actionable comments posted: 0
🧹 Nitpick comments (2)
apps/web/src/routes/_view/templates/$slug.tsx (1)
27-45: Consider aligning OG/Twitter titles with the new page title suffixYou updated the document title to
${template.title} - Hyprnote Templates, butog:titleandtwitter:titlestill use${template.title} - Meeting Template. If the goal is consistent branding across all title-like tags, consider updating those too:- { - property: "og:title", - content: `${template.title} - Meeting Template`, - }, + { + property: "og:title", + content: `${template.title} - Hyprnote Templates`, + }, @@ - { - name: "twitter:title", - content: `${template.title} - Meeting Template`, - }, + { + name: "twitter:title", + content: `${template.title} - Hyprnote Templates`, + },If you intentionally want different strings for page vs. social titles, a short comment explaining that would help future maintainers.
apps/web/src/routes/_view/changelog/index.tsx (1)
14-29: Changelog index head metadata added; optional naming tweakThe new
headblock wires up sensible defaults (title, description, og tags) and matches patterns used by other index routes. Implementation looks correct.If you want slightly less repetition, you could optionally shorten the title to something like
"Hyprnote Changelog"while keeping the rest as is, but that’s purely a branding choice, not a code issue.
📜 Review details
Configuration used: Path: .coderabbit.yaml
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (7)
apps/web/src/routes/_view/blog/$slug.tsx(1 hunks)apps/web/src/routes/_view/blog/index.tsx(1 hunks)apps/web/src/routes/_view/changelog/$slug.tsx(1 hunks)apps/web/src/routes/_view/changelog/index.tsx(1 hunks)apps/web/src/routes/_view/docs/$.tsx(1 hunks)apps/web/src/routes/_view/templates/$slug.tsx(1 hunks)apps/web/src/routes/_view/templates/index.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, usecn(import from@hypr/utils). It is similar toclsx. Always pass an array and split by logical grouping.
Usemotion/reactinstead offramer-motion.
Files:
apps/web/src/routes/_view/templates/index.tsxapps/web/src/routes/_view/blog/$slug.tsxapps/web/src/routes/_view/templates/$slug.tsxapps/web/src/routes/_view/changelog/index.tsxapps/web/src/routes/_view/changelog/$slug.tsxapps/web/src/routes/_view/blog/index.tsxapps/web/src/routes/_view/docs/$.tsx
🧠 Learnings (2)
📚 Learning: 2025-11-24T16:32:29.314Z
Learnt from: CR
Repo: fastrepl/hyprnote PR: 0
File: apps/web/content/changelog/AGENTS.md:0-0
Timestamp: 2025-11-24T16:32:29.314Z
Learning: Applies to apps/web/content/changelog/** : Only keep desktop-related changes when maintaining changelog entries from commits and diffs
Applied to files:
apps/web/src/routes/_view/changelog/$slug.tsx
📚 Learning: 2025-11-24T16:32:30.770Z
Learnt from: CR
Repo: fastrepl/hyprnote PR: 0
File: apps/web/content/changelog/AGENTS.md:0-0
Timestamp: 2025-11-24T16:32:30.770Z
Learning: Applies to apps/web/content/changelog/** : Only include desktop-related changes in the changelog when reading through commits and diffs
Applied to files:
apps/web/src/routes/_view/changelog/$slug.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: ci (macos, macos-14)
- GitHub Check: fmt
🔇 Additional comments (5)
apps/web/src/routes/_view/blog/index.tsx (1)
37-45: Blog index meta title suffix looks good and consistentThe updated
titleandog:titleto"Blog - Hyprnote Blog"align with the new branding pattern and with the post page metadata. No issues spotted.apps/web/src/routes/_view/templates/index.tsx (1)
24-36: Templates index meta titles updated consistentlyThe
titleandog:titlenow use"Meeting Templates - Hyprnote Templates", which matches the new naming scheme and keeps index metadata consistent with the rest of the app.apps/web/src/routes/_view/blog/$slug.tsx (1)
58-70: Post-level meta titles now consistently use “ - Hyprnote Blog”
title,og:title, andtwitter:titleall now append" - Hyprnote Blog", which matches the index page and gives consistent SEO/social titles. Looks good.apps/web/src/routes/_view/docs/$.tsx (1)
51-69: Docs metadata titles correctly standardized
title,og:title, andtwitter:titlenow consistently use${doc.title} - Hyprnote Documentation, with descriptions unchanged. This cleanly aligns docs pages with the new SEO pattern.apps/web/src/routes/_view/changelog/$slug.tsx (1)
63-83: Version page title reordered for clearer SEOComputing
titleasVersion ${changelog.version} - Hyprnote Changelogimproves scanability (version first) and keeps all downstream meta tags (og/twitter) in sync via the sharedtitlevariable. No issues.
|
The latest updates on your projects. Learn more about Argos notifications ↗︎
|
Summary
Updates meta titles across the web app to follow a consistent format:
"Page Title - Hyprnote [Section]". This affects templates, docs, blog, and changelog pages.Changes by section:
"Meeting Templates - Hyprnote Templates"and"${template.title} - Hyprnote Templates""${doc.title} - Hyprnote Documentation""Blog - Hyprnote Blog"and"${article.title} - Hyprnote Blog"headfunction with"Changelog - Hyprnote Changelog"and"Version ${version} - Hyprnote Changelog"Also updates corresponding
og:titleandtwitter:titlemeta tags for consistency.Review & Testing Checklist for Human
Recommended test plan: Deploy to preview environment and manually check browser tabs on:
/templatesand/templates/[any-slug]/docs/[any-slug]/blogand/blog/[any-slug]/changelogand/changelog/[any-slug]Notes