Unify demo naming, live URL, and consultation CTA on marketing pages#122
Unify demo naming, live URL, and consultation CTA on marketing pages#122justin808 wants to merge 1 commit into
Conversation
…ages Marketing-page and site-config half of a docs/marketing consistency pass. The docs content itself lives in the shakacode/react_on_rails monorepo (synced + gitignored here), so those fixes are tracked in a separate issue. Demo naming and live URL (/pro): - Rename the flagship RSC demo heading "LocalHub - RSC performance demo" to "Marketplace - RSC performance demo", matching demos.ts and resolving the heading/body contradiction (heading said LocalHub, body said "a marketplace built on Pro"). - Point the live-demo link at the root https://rsc.reactonrails.com instead of the /search-performance deep link. Consultation CTA consolidation (one label + one destination everywhere): - Label is now "Book a free 30-min call" in the navbar, announcement banner, footer link, per-doc footer card, homepage, and /pro page. - Destination is the HubSpot meeting link everywhere; the navbar CTA no longer points at shakacode.com/contact. - Drop the "complimentary"/"assessment" wording variants. No layout or styling changes; copy and one stale CSS comment only. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
|
Warning Review limit reached
More reviews will be available in 7 minutes and 38 seconds. Learn how PR review limits work. Your organization has run out of usage credits. Purchase more in the billing tab. ⌛ How to resolve this issue?After more reviews become available, a review can be triggered using the We recommend that you space out your commits to avoid hitting the rate limit. 🚦 How do rate limits work?CodeRabbit enforces hourly rate limits for each developer per organization. Our paid plans include higher PR review limits than trial, open-source, and free plans. In all cases, reviews become available again over time. During sustained high-volume PR review activity, CodeRabbit may temporarily slow when the next review becomes available. Please see our Fair Usage Limits Policy for further information. ℹ️ Review info⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (5)
✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
Greptile SummaryThis PR unifies the consultation CTA label to "Book a free 30-min call" pointing to the HubSpot meeting link across all six marketing surfaces, and corrects the
Confidence Score: 4/5Safe to merge — all changes are copy/label text with no logic, routing, or data-flow modifications. Every changed line is a string literal or a CSS comment. The only nit is that the DocItem footer card body text says "30-minute" (spelled out) while the link label immediately below says "30-min" (abbreviated), leaving a small inconsistency inside that card despite the PR's goal of uniform wording. prototypes/docusaurus/src/theme/DocItem/Footer/index.tsx — body text and link label use different forms of "30-min/30-minute" Important Files Changed
Flowchart%%{init: {'theme': 'neutral'}}%%
flowchart TD
A["Announcement Bar\n'Book a free 30-min call'"] --> Z
B["Navbar CTA\n'Book a free 30-min call'\n(was: Get Expert Help → shakacode.com/contact)"] --> Z
C["Footer Link\n'Book a free 30-min call'"] --> Z
D["Homepage CTA\n'Book a free 30-min call'"] --> Z
E["/pro Page CTA\n'Book a free 30-min call'"] --> Z
F["DocItem Footer Card\n'Book a free 30-min call'"] --> Z
Z["https://meetings.hubspot.com/justingordon/30-minute-consultation"]
G["/pro Demo Card\nMarketplace — RSC performance demo"] --> H["https://rsc.reactonrails.com\n(was: /search-performance)"]
Reviews (1): Last reviewed commit: "Unify demo naming, live demo URL, and co..." | Re-trigger Greptile |
| ShakaCode builds and maintains React on Rails. Book a free 30-minute call | ||
| to get hands-on advice about your architecture, performance, or migration path. |
There was a problem hiding this comment.
The body sentence uses "30-minute" (spelled out) while the link immediately below uses "30-min" (abbreviated). Since the stated goal of this PR is one consistent label everywhere, aligning the body text to "30-min" as well keeps the card internally consistent.
| ShakaCode builds and maintains React on Rails. Book a free 30-minute call | |
| to get hands-on advice about your architecture, performance, or migration path. | |
| ShakaCode builds and maintains React on Rails. Book a free 30-min call | |
| to get hands-on advice about your architecture, performance, or migration path. |
Note: If this suggestion doesn't match your team's coding style, reply to this and let me know. I'll remember it for next time!
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 17797bcc32
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
| id: 'consultation_cta', | ||
| content: | ||
| 'Want expert advice on your React on Rails setup? <a href="https://meetings.hubspot.com/justingordon/30-minute-consultation">Book a complimentary 30-minute assessment</a> with the ShakaCode team.', | ||
| 'Want expert advice on your React on Rails setup? <a href="https://meetings.hubspot.com/justingordon/30-minute-consultation">Book a free 30-min call</a> with the ShakaCode team.', |
There was a problem hiding this comment.
Bump the announcement id for the new CTA
For users who previously closed the old announcement bar, Docusaurus persists dismissal by announcementBar.id; since this changes the CTA copy but keeps the existing consultation_cta id, those returning users will never see the new unified CTA. If the updated consultation message is intended to be shown after this copy change, use a new id for this announcement.
Useful? React with 👍 / 👎.
|
Cloudflare preview deployed.
|
What
The marketing-page + site-config half of a docs/marketing consistency pass. Unifies the flagship demo name, its live URL, and the consultation CTA across every surface in this repo.
The docs half (introduction, tutorial, README, changelog link, version floors, v15 retraction notice) lives in the
shakacode/react_on_railsmonorepo — docs are synced + gitignored here — and is tracked in shakacode/react_on_rails#3596.Changes (copy + config only — no layout/styling changes)
Demo naming + live URL (
/pro)LocalHub — RSC performance demoMarketplace — RSC performance demorsc.reactonrails.com/search-performancersc.reactonrails.com(root)This also fixes the contradiction where the heading said "LocalHub" but the body said "a marketplace built on Pro."
demos.tswas already "Marketplace".Consultation CTA — one label, one destination
Label
Book a free 30-min calland destinationhttps://meetings.hubspot.com/justingordon/30-minute-consultationeverywhere:Get Expert Helpshakacode.com/contact← only one that pointed elsewhereBook a complimentary 30-minute assessmentBook a Complimentary AssessmentBook a free callBook a free call/propageBook a 30-minute consultationAll →
Book a free 30-min call→ HubSpot meeting link.Already consistent (no change needed): legacy demo URL in
demos.tswas alreadyhttps://www.reactrails.com; the footer Changelog link already targets/docs/upgrading/changelog(verified it exists and resolves); the Gumroad "coming soon" card was already replaced with a live URL + screenshot in a prior commit.Verification
npm run build→[SUCCESS]. Broken-link warnings are all in upstream/docs/*synced content and pre-existing; none come from these changes — no marketing page, footer, or nav link appears in the broken list, and the Changelog link resolves.LocalHub,search-performance,Get Expert Help,Complimentary,shakacode.com/contact, or barereactrails.comremain.These are text/label changes with no layout or CSS-visual impact (one stale CSS comment updated), so the before/after above is the full visual delta. Happy to attach screenshots or a preview-deploy walkthrough if you'd like them.
Follow-up
🤖 Generated with Claude Code
Note
Low Risk
Text and link updates in marketing/config only; no application logic, auth, or data handling.
Overview
This PR aligns marketing copy and site config in the Docusaurus prototype: no layout or styling changes beyond a CSS comment tweak.
On
/pro, the flagship demo is renamed from LocalHub to Marketplace — RSC performance demo, and the live link moves fromrsc.reactonrails.com/search-performancetohttps://rsc.reactonrails.com, matching the marketplace description and other demo data.Consultation CTAs are standardized to the label
Book a free 30-min calland the HubSpot meeting URL everywhere they appear: announcement bar, navbar (replacing Get Expert Help →shakacode.com/contact), footer, homepage,/pro, and the per-doc footer callout. Wording shifts from variants like “complimentary assessment” and “Book a free call” to the single label.Reviewed by Cursor Bugbot for commit 17797bc. Bugbot is set up for automated code reviews on this repo. Configure here.