Skip to content

Conversation

@NicholasKissel
Copy link
Member

Changes

Copy link
Member Author

NicholasKissel commented May 13, 2025

Warning

This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more


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.

@NicholasKissel NicholasKissel marked this pull request as ready for review May 13, 2025 20:16
Copy link

@greptile-apps greptile-apps bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

PR Summary

This PR significantly revamps the marketing landing page with a more focused and modern design approach.

  • Streamlined hero section with centered layout, prominent CTAs, and Docker command for local deployment
  • Added new Feature component with hover effects and contextual use cases for each capability
  • Implemented comprehensive FrameworksSection showcasing 15 supported technologies with interactive icons
  • Removed several complex sections (PowerfulPrimitives, ServerlessLimitations, RivetCloud) in favor of clearer, more direct content
  • Enhanced SEO with updated metadata focusing on "Open-Source Serverless Platform" positioning

The changes create a more cohesive and user-friendly landing experience while better highlighting Rivet's key capabilities and framework support.

1 file(s) reviewed, 3 comment(s)
Edit PR Review Bot Settings | Greptile

Comment on lines +239 to +241
faIcon: faServer,
href: "/docs/stateful-jobs",
useCases: ["Code interpreters", "Remote desktop", "UGC games"]
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

logic: Both 'Sandboxed Containers' and 'Stateful Actors' link to /docs/stateful-jobs - verify if this is intentional or if Containers should have its own documentation page

</p>
</div>
<div className="md:w-2/3">
<div className="grid grid-cols-4 md:grid-cols-5 lg:grid-cols-6 gap-x-6 gap-y-6">
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

style: Grid layout starts at 4 columns on mobile which may be too wide for small screens. Consider starting at 2 or 3 columns

Comment on lines +99 to +101
<Link
href="#deploy"
className="flex items-center justify-center relative"
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

logic: Empty #deploy anchor link - ensure this section exists or update the href

@cloudflare-workers-and-pages
Copy link

cloudflare-workers-and-pages bot commented May 13, 2025

Deploying rivet with  Cloudflare Pages  Cloudflare Pages

Latest commit: b7907cb
Status: ✅  Deploy successful!
Preview URL: https://9102f591.rivet.pages.dev
Branch Preview URL: https://05-13-chore-pricing-page.rivet.pages.dev

View logs

@NicholasKissel NicholasKissel force-pushed the 05-13-chore_pricing_page branch 2 times, most recently from f98de53 to 905bbca Compare May 14, 2025 07:14
@NathanFlurry NathanFlurry force-pushed the 05-13-chore_pricing_page branch from 905bbca to b7907cb Compare May 19, 2025 17:14
@NathanFlurry NathanFlurry force-pushed the 04-29-chore_fill_out_remaining_content_for_landing_page branch from d4f2bbf to 05aaf76 Compare May 19, 2025 17:14
@NathanFlurry NathanFlurry changed the base branch from 04-29-chore_fill_out_remaining_content_for_landing_page to graphite-base/2441 May 19, 2025 17:59
@NathanFlurry NathanFlurry force-pushed the 05-13-chore_pricing_page branch from b7907cb to a9e634b Compare May 19, 2025 18:08
Copy link
Member

This PR was a mistake

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.

3 participants