Skip to content

fix(landing): rename /comparison route to /comparisons for naming consistency#5651

Merged
waleedlatif1 merged 1 commit into
stagingfrom
comparisons-route-rename
Jul 13, 2026
Merged

fix(landing): rename /comparison route to /comparisons for naming consistency#5651
waleedlatif1 merged 1 commit into
stagingfrom
comparisons-route-rename

Conversation

@waleedlatif1

Copy link
Copy Markdown
Collaborator

Summary

  • Renames the /comparison route to /comparisons (e.g. /comparison/tines/comparisons/tines), aligning with the /integrations/[slug] convention (plural category, singular item)
  • Adds a 301 redirect for both the bare hub page and every competitor detail page, so previously indexed/bookmarked URLs keep working
  • Updates every internal reference: footer nav link, sitemap generation, and all import specifiers/canonical URLs/breadcrumb JSON-LD within the route itself

Type of Change

  • Bug fix (naming consistency / info architecture)

Testing

  • bunx tsc --noEmit clean
  • bun run lint clean
  • Real production build + verified live:
    • /comparisons and /comparisons/tines return 200
    • /comparison and /comparison/tines 308-redirect to the new paths
    • Footer "Compare" link, sitemap.xml entries, and breadcrumb/ItemList JSON-LD all point at /comparisons

Checklist

  • Code follows project style guidelines
  • Self-reviewed my changes
  • Tests added/updated and passing
  • No new warnings introduced
  • I confirm that I have read and agree to the terms outlined in the Contributor License Agreement (CLA)

…sistency

Aligns with the /integrations/[slug] convention (plural category, singular
item). Adds a 301 redirect (both the bare hub page and every competitor
detail page) so previously indexed URLs keep working, and updates every
internal reference: footer nav, sitemap generation, and all import
specifiers/canonical URLs/breadcrumb JSON-LD within the route itself.
@vercel

vercel Bot commented Jul 13, 2026

Copy link
Copy Markdown

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

1 Skipped Deployment
Project Deployment Actions Updated (UTC)
docs Skipped Skipped Jul 13, 2026 11:51pm

Request Review

@waleedlatif1

Copy link
Copy Markdown
Collaborator Author

@greptile

@waleedlatif1

Copy link
Copy Markdown
Collaborator Author

@cursor review

@cursor

cursor Bot commented Jul 13, 2026

Copy link
Copy Markdown

PR Summary

Low Risk
Landing URL and import-path rename with permanent redirects; no auth, data, or business-logic changes.

Overview
Renames the landing comparison hub and competitor pages from /comparison to /comparisons (e.g. /comparisons/tines), matching the plural category pattern used elsewhere (e.g. /integrations).

Permanent 301 redirects in next.config.ts map /comparison and /comparison/:path* to the new paths so bookmarks and indexed URLs keep working.

Internal references are updated to the new URL and module path under app/(landing)/comparisons/: footer Compare link, sitemap entries, canonical/metadata paths, in-page links, breadcrumb/JSON-LD, and import specifiers in comparison components.

Reviewed by Cursor Bugbot for commit 621302b. Configure here.

@cursor cursor 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.

✅ Bugbot reviewed your changes and found no new issues!

Comment @cursor review or bugbot run to trigger another review on this PR

Reviewed by Cursor Bugbot for commit 621302b. Configure here.

@greptile-apps

greptile-apps Bot commented Jul 13, 2026

Copy link
Copy Markdown
Contributor

Greptile Summary

This PR renames the landing comparison route to the plural /comparisons path. The main changes are:

  • Moves the landing comparison pages and components under comparisons.
  • Updates internal imports, links, metadata, JSON-LD, and sitemap URLs.
  • Adds permanent redirects from the old /comparison URLs to the new paths.

Confidence Score: 5/5

This looks safe to merge.

  • No blocking issues found in the changed code.

Important Files Changed

Filename Overview
apps/sim/next.config.ts Adds permanent redirects from the old singular comparison paths to the new plural paths.
apps/sim/app/sitemap.ts Updates comparison sitemap entries to use /comparisons.
apps/sim/app/(landing)/comparisons/page.tsx Updates the comparison hub route, canonical metadata, JSON-LD, and links.
apps/sim/app/(landing)/comparisons/[provider]/page.tsx Updates the provider comparison route, imports, metadata, breadcrumbs, JSON-LD, and back link.

Reviews (3): Last reviewed commit: "fix(landing): rename /comparison route t..." | Re-trigger Greptile

Comment thread apps/sim/next.config.ts
@greptile-apps

greptile-apps Bot commented Jul 13, 2026

Copy link
Copy Markdown
Contributor

Greptile Summary

This PR renames the landing comparison route to /comparisons. The main changes are:

  • Moves the comparison hub and detail pages under the plural route.
  • Updates imports, links, metadata, JSON-LD, and sitemap URLs.
  • Adds redirects from the old /comparison paths to the new URLs.

Confidence Score: 5/5

This looks safe to merge.

  • No blocking issues found in the changed code.

Important Files Changed

Filename Overview
apps/sim/next.config.ts Adds permanent redirects for the old comparison hub and detail paths.
apps/sim/app/(landing)/comparisons/page.tsx Updates the comparison hub page imports, links, metadata, and structured data to the plural route.
apps/sim/app/(landing)/comparisons/[provider]/page.tsx Updates the comparison detail page imports, canonical paths, breadcrumbs, structured data, and back link.
apps/sim/app/sitemap.ts Updates sitemap generation to emit /comparisons URLs.
apps/sim/app/(landing)/components/footer/footer.tsx Updates the footer Compare link to the new route.

Reviews (2): Last reviewed commit: "fix(landing): rename /comparison route t..." | Re-trigger Greptile

@waleedlatif1

Copy link
Copy Markdown
Collaborator Author

@greptile

@waleedlatif1 waleedlatif1 merged commit 5b5dd19 into staging Jul 13, 2026
13 checks passed
@waleedlatif1 waleedlatif1 deleted the comparisons-route-rename branch July 13, 2026 23:54
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