Skip to content

Conversation

@NicholasKissel
Copy link
Member

No description provided.

@vercel
Copy link

vercel bot commented Dec 10, 2025

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

Project Deployment Preview Comments Updated (UTC)
rivet-site Ready Ready Preview Comment Dec 10, 2025 2:13am
3 Skipped Deployments
Project Deployment Preview Comments Updated (UTC)
rivet-cloud Ignored Ignored Dec 10, 2025 2:13am
rivet-inspector Ignored Ignored Preview Dec 10, 2025 2:13am
rivetkit-serverless Skipped Skipped Dec 10, 2025 2:13am

@vercel vercel bot temporarily deployed to Preview – rivetkit-serverless December 10, 2025 02:09 Inactive
Copy link
Member Author


How to use the Graphite Merge Queue

Add the label merge-queue to this PR to add it to the merge queue.

You must have a Graphite account in order to use the merge queue. Sign up using this link.

An organization admin has enabled the Graphite Merge Queue in this repository.

Please do not merge from GitHub as this will restart CI on PRs being processed by the merge queue.

This stack of pull requests is managed by Graphite. Learn more about stacking.

@claude
Copy link

claude bot commented Dec 10, 2025

PR Review: Footer Pricing Link Update

Summary

This PR updates the pricing link in the website footer from /pricing to /cloud#pricing, directing users to a specific section on the cloud page.

Code Quality ✅

  • Clean change: Simple, focused one-line update
  • Follows conventions: Properly formatted JSX
  • Commit message: Follows conventional commit format with chore(site): prefix

Functional Review ✅

  • Link structure: The change from /pricing/cloud#pricing uses a proper hash anchor
  • Consistency check: ✅ This appears to be the only "Pricing" link in the footer (line 38)

Recommendations

  1. Verify destination page: Ensure that:

    • The /cloud page exists and is accessible
    • There's a corresponding element with id="pricing" or similar anchor on that page
    • The anchor scrolls to the correct pricing section
  2. Check for other pricing references: Consider searching the codebase for other pricing links that may need similar updates

  3. Testing checklist:

    • Link navigates to /cloud page
    • Page scrolls to pricing section on load
    • Anchor works correctly in all target browsers

Security & Performance ✅

  • No security concerns (internal navigation link)
  • No performance impact

Test Coverage

  • Note: This is a static link change in JSX, so automated tests may not be necessary. Manual verification of the link destination is recommended.

Verdict

LGTM pending verification that the /cloud#pricing anchor exists and functions correctly on the target page. The code change itself is clean and straightforward.

@pkg-pr-new
Copy link

pkg-pr-new bot commented Dec 10, 2025

More templates

@rivetkit/cloudflare-workers

pnpm add https://pkg.pr.new/rivet-dev/rivet/@rivetkit/cloudflare-workers@3616

@rivetkit/db

pnpm add https://pkg.pr.new/rivet-dev/rivet/@rivetkit/db@3616

@rivetkit/framework-base

pnpm add https://pkg.pr.new/rivet-dev/rivet/@rivetkit/framework-base@3616

@rivetkit/next-js

pnpm add https://pkg.pr.new/rivet-dev/rivet/@rivetkit/next-js@3616

@rivetkit/react

pnpm add https://pkg.pr.new/rivet-dev/rivet/@rivetkit/react@3616

rivetkit

pnpm add https://pkg.pr.new/rivet-dev/rivet/rivetkit@3616

@rivetkit/sql-loader

pnpm add https://pkg.pr.new/rivet-dev/rivet/@rivetkit/sql-loader@3616

@rivetkit/engine-runner

pnpm add https://pkg.pr.new/rivet-dev/rivet/@rivetkit/engine-runner@3616

@rivetkit/engine-runner-protocol

pnpm add https://pkg.pr.new/rivet-dev/rivet/@rivetkit/engine-runner-protocol@3616

commit: 223107c

@graphite-app
Copy link
Contributor

graphite-app bot commented Dec 10, 2025

Merge activity

  • Dec 10, 7:41 PM UTC: NicholasKissel added this pull request to the Graphite merge queue.
  • Dec 10, 7:42 PM UTC: CI is running for this pull request on a draft pull request (#3621) due to your merge queue CI optimization settings.
  • Dec 10, 7:43 PM UTC: Merged by the Graphite merge queue via draft PR: #3621.

graphite-app bot pushed a commit that referenced this pull request Dec 10, 2025
@graphite-app graphite-app bot closed this Dec 10, 2025
@graphite-app graphite-app bot deleted the 12-09-chore_site_footer_pricing_link_update branch December 10, 2025 19:43
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