docs: replace dead pro.reactonrails.com links#2851
Merged
Conversation
|
You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard. |
Contributor
WalkthroughThis PR systematically updates Pro-related documentation URLs across the codebase. The primary change redirects URLs from Changes
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~3 minutes Possibly related PRs
Suggested labels
Poem
🚥 Pre-merge checks | ✅ 3✅ Passed checks (3 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches📝 Generate docstrings
🧪 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 is a targeted link-maintenance PR that replaces all dead Confidence Score: 5/5Safe to merge — purely mechanical URL replacements with no logic changes and no test regressions. All changes are one-for-one URL substitutions. The author validated URLs with curl, ran rubocop and prettier, and scoped the PR to avoid conflicts with other open PRs. The only Ruby change does not break any existing tests. No files require special attention. Important Files Changed
Flowchart%%{init: {'theme': 'neutral'}}%%
flowchart TD
A["pro.reactonrails.com (HTTP 500 ❌)"] -->|"replaced with"| B["reactonrails.com/docs/pro/ (HTTP 200 ✅)"]
B --> C["README.md (3 links)"]
B --> D["docs/oss/**/*.md (17 links)"]
B --> E["docs/pro/**/*.md (2 links)"]
B --> F["react_on_rails/lib/react_on_rails/utils.rb (1 runtime warning)"]
Reviews (1): Last reviewed commit: "docs: replace dead Pro CTA links" | Re-trigger Greptile |
Contributor
|
The changes look correct and well-scoped. All updated links are consistent, the Ruby warning string update is accurate, and the PR description clearly documents the intentional exclusions. Remaining dead links not covered by this PR A few files outside the stated exclusions still carry
If these are already covered by the active PRs listed in the scope notes, calling that out explicitly would help reviewers confirm nothing is falling through the cracks. Minor: link text vs URL consistency Several places use generic link text like "Get a license" that now points to the general docs page rather than a purchase/contact flow. Fine for now, but worth revisiting once the Pro site structure is finalised. Overall: LGTM - fix is correct and the validation steps in the PR description are thorough. |
justin808
commented
Mar 26, 2026
| # Node Renderer Debugging | ||
|
|
||
| > **Pro Feature** — Available with [React on Rails Pro](https://pro.reactonrails.com). | ||
| > Free or very low cost for startups and small companies. [Get a license →](https://pro.reactonrails.com) |
Member
Author
There was a problem hiding this comment.
in every one of these blocks the first one should be the docs link, so that's the one to change and the second one is the license link
Member
Author
There was a problem hiding this comment.
Updated in 4d63e2f. The first link in each block now stays on the docs page, and the second Get a license CTA is back to the license URL.
justin808
added a commit
that referenced
this pull request
Mar 26, 2026
## Summary - replace stale generator/help links that still point at moved docs or `blob/master` paths - update scaffolded initializer comments to the current published configuration/testing docs - refresh the manual-render example page so its code-splitting link points at the live docs page ## Scope notes - Limited to generator/help/example URLs only - Intentionally separate from #2850 and #2851 to keep review small - No runtime behavior changes ## Validation Passed: - `bundle exec rubocop react_on_rails/lib/react_on_rails/test_helper/webpack_assets_compiler.rb react_on_rails/lib/react_on_rails/helper.rb react_on_rails/lib/generators/react_on_rails/templates/base/base/config/initializers/react_on_rails.rb.tt` - `git diff --check` - `curl -I -L` checks showing `200` for: - `https://reactonrails.com/docs/api-reference/generator-details/` - `https://reactonrails.com/docs/building-features/dev-server-and-testing/` - `https://reactonrails.com/docs/building-features/testing-configuration/` - `https://reactonrails.com/docs/building-features/code-splitting/` - `https://github.com/shakacode/react_on_rails/blob/main/react_on_rails/spec/dummy/client/app/startup/ReactHelmetApp.server.jsx` <!-- CURSOR_SUMMARY --> --- > [!NOTE] > **Low Risk** > Low risk: updates only documentation links in generator templates, helper error messages, and dummy/example views, with no runtime logic changes. > > **Overview** > Refreshes several stale documentation URLs to point at the current `reactonrails.com` docs, including generator usage output and the scaffolded `config/initializers/react_on_rails.rb` template comments. > > Also updates in-code guidance links in `ReactOnRails::Helper` error messages and `TestHelper::WebpackAssetsCompiler`, plus the dummy manual-render example page’s code-splitting link, to avoid `blob/master` and moved-doc paths. > > <sup>Written by [Cursor Bugbot](https://cursor.com/dashboard?tab=bugbot) for commit f1afee9. This will update automatically on new commits. Configure [here](https://cursor.com/dashboard?tab=bugbot).</sup> <!-- /CURSOR_SUMMARY -->
![claude[bot]](https://avatars.githubusercontent.com/in/1236702?s=60&v=4){kind=small}
|
|
||
| > **Pro Feature** — Available with [React on Rails Pro](https://pro.reactonrails.com). | ||
| > Free or very low cost for startups and small companies. [Get a license →](https://pro.reactonrails.com) | ||
| > **Pro Feature** — Available with [React on Rails Pro](https://reactonrails.com/docs/pro/). |
Contributor
There was a problem hiding this comment.
The "Get a license →" link was only updated with a trailing slash but still points to pro.reactonrails.com, which the PR confirms returns HTTP 500. This should be updated to https://reactonrails.com/docs/pro/ like the first link on the previous line.
Suggested change
| > **Pro Feature** — Available with [React on Rails Pro](https://reactonrails.com/docs/pro/). | |
| > Free or very low cost for startups and small companies. [Get a license →](https://reactonrails.com/docs/pro/) |
|
|
||
| > **Pro Feature** — Available with [React on Rails Pro](https://pro.reactonrails.com). | ||
| > Free or very low cost for startups and small companies. [Get a license →](https://pro.reactonrails.com) | ||
| > **Pro Feature** — Available with [React on Rails Pro](https://reactonrails.com/docs/pro/). |
Contributor
There was a problem hiding this comment.
Same issue — "Get a license →" still points to the broken pro.reactonrails.com domain (only a trailing slash was added).
Suggested change
| > **Pro Feature** — Available with [React on Rails Pro](https://reactonrails.com/docs/pro/). | |
| > Free or very low cost for startups and small companies. [Get a license →](https://reactonrails.com/docs/pro/) |
| @@ -1,7 +1,7 @@ | |||
| # SSR Caching: Prerender Caching and Fragment Caching | |||
|
|
|||
| :::tip Pro Feature | |||
Contributor
There was a problem hiding this comment.
Same issue — "Get a license →" still points to the broken pro.reactonrails.com domain.
Suggested change
| :::tip Pro Feature | |
| Available with [React on Rails Pro](https://reactonrails.com/docs/pro/). Free or very low cost for startups and small companies. [Get a license →](https://reactonrails.com/docs/pro/) |
|
|
||
| > **Pro Feature** — Available with [React on Rails Pro](https://pro.reactonrails.com). | ||
| > Free or very low cost for startups and small companies. [Get a license →](https://pro.reactonrails.com) | ||
| > **Pro Feature** — Available with [React on Rails Pro](https://reactonrails.com/docs/pro/). |
Contributor
There was a problem hiding this comment.
Same issue — "Get a license →" still points to the broken pro.reactonrails.com domain.
Suggested change
| > **Pro Feature** — Available with [React on Rails Pro](https://reactonrails.com/docs/pro/). | |
| > Free or very low cost for startups and small companies. [Get a license →](https://reactonrails.com/docs/pro/) |
|
|
||
| > **Pro Feature** — Available with [React on Rails Pro](https://pro.reactonrails.com). | ||
| > Free or very low cost for startups and small companies. [Get a license →](https://pro.reactonrails.com) | ||
| > **Pro Feature** — Available with [React on Rails Pro](https://reactonrails.com/docs/pro/). |
Contributor
There was a problem hiding this comment.
Same issue — "Get a license →" still points to the broken pro.reactonrails.com domain.
Suggested change
| > **Pro Feature** — Available with [React on Rails Pro](https://reactonrails.com/docs/pro/). | |
| > Free or very low cost for startups and small companies. [Get a license →](https://reactonrails.com/docs/pro/) |
|
|
||
| > **Pro Feature** — Available with [React on Rails Pro](https://pro.reactonrails.com). | ||
| > Free or very low cost for startups and small companies. [Get a license →](https://pro.reactonrails.com) | ||
| > **Pro Feature** — Available with [React on Rails Pro](https://reactonrails.com/docs/pro/). |
Contributor
There was a problem hiding this comment.
Same issue — "Get a license →" still points to the broken pro.reactonrails.com domain.
Suggested change
| > **Pro Feature** — Available with [React on Rails Pro](https://reactonrails.com/docs/pro/). | |
| > Free or very low cost for startups and small companies. [Get a license →](https://reactonrails.com/docs/pro/) |
|
|
||
| > **Pro Feature** — Available with [React on Rails Pro](https://pro.reactonrails.com). | ||
| > Free or very low cost for startups and small companies. [Get a license →](https://pro.reactonrails.com) | ||
| > **Pro Feature** — Available with [React on Rails Pro](https://reactonrails.com/docs/pro/). |
Contributor
There was a problem hiding this comment.
Same issue — "Get a license →" still points to the broken pro.reactonrails.com domain.
Suggested change
| > **Pro Feature** — Available with [React on Rails Pro](https://reactonrails.com/docs/pro/). | |
| > Free or very low cost for startups and small companies. [Get a license →](https://reactonrails.com/docs/pro/) |
|
|
||
| > **Pro Feature** — Available with [React on Rails Pro](https://pro.reactonrails.com). | ||
| > Free or very low cost for startups and small companies. [Get a license →](https://pro.reactonrails.com) | ||
| > **Pro Feature** — Available with [React on Rails Pro](https://reactonrails.com/docs/pro/). |
Contributor
There was a problem hiding this comment.
Same issue — "Get a license →" still points to the broken pro.reactonrails.com domain.
Suggested change
| > **Pro Feature** — Available with [React on Rails Pro](https://reactonrails.com/docs/pro/). | |
| > Free or very low cost for startups and small companies. [Get a license →](https://reactonrails.com/docs/pro/) |
|
|
||
| > **Pro Feature** — Available with [React on Rails Pro](https://pro.reactonrails.com). | ||
| > Free or very low cost for startups and small companies. [Get a license →](https://pro.reactonrails.com) | ||
| > **Pro Feature** — Available with [React on Rails Pro](https://reactonrails.com/docs/pro/). |
Contributor
There was a problem hiding this comment.
Same issue — "Get a license →" still points to the broken pro.reactonrails.com domain.
Suggested change
| > **Pro Feature** — Available with [React on Rails Pro](https://reactonrails.com/docs/pro/). | |
| > Free or very low cost for startups and small companies. [Get a license →](https://reactonrails.com/docs/pro/) |
|
|
||
| > **Pro Feature** — Available with [React on Rails Pro](https://pro.reactonrails.com). | ||
| > Free or very low cost for startups and small companies. [Get a license →](https://pro.reactonrails.com) | ||
| > **Pro Feature** — Available with [React on Rails Pro](https://reactonrails.com/docs/pro/). |
Contributor
There was a problem hiding this comment.
Same issue — "Get a license →" still points to the broken pro.reactonrails.com domain.
Suggested change
| > **Pro Feature** — Available with [React on Rails Pro](https://reactonrails.com/docs/pro/). | |
| > Free or very low cost for startups and small companies. [Get a license →](https://reactonrails.com/docs/pro/) |
|
|
||
| > **Pro Feature** — React Server Components require [React on Rails Pro](https://pro.reactonrails.com/) 4+ with the node renderer. | ||
| > Free or very low cost for startups and small companies. [Get a license →](https://pro.reactonrails.com) | ||
| > **Pro Feature** — React Server Components require [React on Rails Pro](https://reactonrails.com/docs/pro/) 4+ with the node renderer. |
Contributor
There was a problem hiding this comment.
Same issue — "Get a license →" still points to the broken pro.reactonrails.com domain.
Suggested change
| > **Pro Feature** — React Server Components require [React on Rails Pro](https://reactonrails.com/docs/pro/) 4+ with the node renderer. | |
| > Free or very low cost for startups and small companies. [Get a license →](https://reactonrails.com/docs/pro/) |
coderabbitai
Bot
added
review-needed
documentation
docs-cleanup
Contributor
|
Review: Incomplete link replacement for Get-a-license links The overall goal is correct and the approach is solid. Good call fixing the runtime warning in utils.rb too. One consistent gap: In 11 files that use the two-line Pro feature callout block, the first link was correctly updated to Pattern seen in this PR across affected files:
Affected files (inline suggestions posted on each):
All suggested fixes point these links to |
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
Contributor
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.
Inline comments:
In `@docs/oss/building-features/node-renderer/basics.md`:
- Around line 3-4: Update the remaining CTA link so it matches the
trailing-slash variant used elsewhere: locate the "Get a license →" anchor (the
Pro Feature CTA) that currently points to "https://pro.reactonrails.com" and
change it to "https://pro.reactonrails.com/" to ensure consistency and avoid the
dead-link variant.
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: Organization UI
Review profile: CHILL
Plan: Pro
Run ID: f89893f3-5926-451b-a0d6-85e9f439793a
📒 Files selected for processing (23)
README.mddocs/oss/api-reference/ruby-api-pro.mddocs/oss/api-reference/view-helpers-api.mddocs/oss/building-features/bundle-caching.mddocs/oss/building-features/caching.mddocs/oss/building-features/code-splitting.mddocs/oss/building-features/node-renderer/basics.mddocs/oss/building-features/node-renderer/debugging.mddocs/oss/building-features/node-renderer/error-reporting-and-tracing.mddocs/oss/building-features/node-renderer/heroku.mddocs/oss/building-features/node-renderer/js-configuration.mddocs/oss/building-features/node-renderer/troubleshooting.mddocs/oss/building-features/rails-webpacker-react-integration-options.mddocs/oss/building-features/turbolinks.mddocs/oss/configuration/configuration-pro.mddocs/oss/getting-started/comparison-with-alternatives.mddocs/oss/getting-started/oss-vs-pro.mddocs/oss/migrating/migrating-to-rsc.mddocs/oss/upgrading/release-notes/16.1.0.mddocs/oss/upgrading/release-notes/16.2.0.mddocs/oss/upgrading/upgrading-react-on-rails.mddocs/pro/react-server-components/index.mdreact_on_rails/lib/react_on_rails/utils.rb
Comment on lines
+3
to
4
| > **Pro Feature** — Available with [React on Rails Pro](https://reactonrails.com/docs/pro/). | ||
| > Free or very low cost for startups and small companies. [Get a license →](https://pro.reactonrails.com) |
Contributor
There was a problem hiding this comment.
Update the remaining Pro CTA URL for consistency and dead-link avoidance.
Line 3 was updated, but Line 4 still points to https://pro.reactonrails.com (without trailing slash), which is inconsistent with the rest of this PR and likely the leftover dead-link variant.
🔧 Proposed fix
-> Free or very low cost for startups and small companies. [Get a license →](https://pro.reactonrails.com)
+> Free or very low cost for startups and small companies. [Get a license →](https://pro.reactonrails.com/)📝 Committable suggestion
‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.
Suggested change
| > **Pro Feature** — Available with [React on Rails Pro](https://reactonrails.com/docs/pro/). | |
| > Free or very low cost for startups and small companies. [Get a license →](https://pro.reactonrails.com) | |
| > **Pro Feature** — Available with [React on Rails Pro](https://reactonrails.com/docs/pro/). | |
| > Free or very low cost for startups and small companies. [Get a license →](https://pro.reactonrails.com/) |
🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed.
In `@docs/oss/building-features/node-renderer/basics.md` around lines 3 - 4,
Update the remaining CTA link so it matches the trailing-slash variant used
elsewhere: locate the "Get a license →" anchor (the Pro Feature CTA) that
currently points to "https://pro.reactonrails.com" and change it to
"https://pro.reactonrails.com/" to ensure consistency and avoid the dead-link
variant.
|
|
||
| > **Pro Feature** — Available with [React on Rails Pro](https://pro.reactonrails.com). | ||
| > **Pro Feature** — Available with [React on Rails Pro](https://reactonrails.com/docs/pro/). | ||
| > Free or very low cost for startups and small companies. [Get a license →](https://pro.reactonrails.com) |
Contributor
There was a problem hiding this comment.
This "Get a license" link was not updated at all — it still points to https://pro.reactonrails.com (no trailing slash, dead domain returning 500). All the other files in this PR at least got a trailing slash added, but this one was missed entirely.
Suggested change
| > Free or very low cost for startups and small companies. [Get a license →](https://pro.reactonrails.com) | |
| > Free or very low cost for startups and small companies. [Get a license →](https://reactonrails.com/docs/pro/) |
| > **Pro Feature** — Available with [React on Rails Pro](https://pro.reactonrails.com). | ||
| > Free or very low cost for startups and small companies. [Get a license →](https://pro.reactonrails.com) | ||
| > **Pro Feature** — Available with [React on Rails Pro](https://reactonrails.com/docs/pro/). | ||
| > Free or very low cost for startups and small companies. [Get a license →](https://pro.reactonrails.com/) |
Contributor
There was a problem hiding this comment.
The first link on line 3 was correctly updated to https://reactonrails.com/docs/pro/, but this "Get a license" link only had a trailing slash added — it still points to the dead pro.reactonrails.com domain (500). The same pattern applies to the "Get a license" line in: bundle-caching.md, caching.md, code-splitting.md, all node-renderer/*.md files, configuration-pro.md, and docs/pro/react-server-components/index.md.
Suggested change
| > Free or very low cost for startups and small companies. [Get a license →](https://pro.reactonrails.com/) | |
| > Free or very low cost for startups and small companies. [Get a license →](https://reactonrails.com/docs/pro/) |
Contributor
|
See inline comments for the key issues found in review. |
justin808
added a commit
that referenced
this pull request
Mar 27, 2026
…olve-2833 * origin/main: docs: fix profiling node renderer command (#2863) generators: point Pro install fallback to upgrade guide (#2868) Add RSC Flight payload optimization guide (Article 7) (#2827) Migrate from deprecated Async::Variable to Async::Promise (#2832) docs: turn pro quick start into a gateway (#2862) Fix upload-assets endpoint duplicating bundles across directories (#2768) docs: fix stale docs links and help URLs (#2850) docs: replace dead pro.reactonrails.com links (#2851) docs: refresh generator and helper URLs (#2852) Add standalone RSC upgrade guide for existing Pro apps (#2831) Raise docs version floor to 16.4.0 in install/demo guidance (#2610) # Conflicts: # CHANGELOG.md
justin808
added a commit
that referenced
this pull request
Mar 27, 2026
…olve-2834 * origin/main: docs: fix profiling node renderer command (#2863) generators: point Pro install fallback to upgrade guide (#2868) Add RSC Flight payload optimization guide (Article 7) (#2827) Migrate from deprecated Async::Variable to Async::Promise (#2832) docs: turn pro quick start into a gateway (#2862) Fix upload-assets endpoint duplicating bundles across directories (#2768) docs: fix stale docs links and help URLs (#2850) docs: replace dead pro.reactonrails.com links (#2851) docs: refresh generator and helper URLs (#2852) # Conflicts: # CHANGELOG.md # react_on_rails/lib/react_on_rails/utils.rb
justin808
added a commit
that referenced
this pull request
Mar 27, 2026
…olve-2849 * origin/main: docs: fix profiling node renderer command (#2863) generators: point Pro install fallback to upgrade guide (#2868) Add RSC Flight payload optimization guide (Article 7) (#2827) Migrate from deprecated Async::Variable to Async::Promise (#2832) docs: turn pro quick start into a gateway (#2862) Fix upload-assets endpoint duplicating bundles across directories (#2768) docs: fix stale docs links and help URLs (#2850) docs: replace dead pro.reactonrails.com links (#2851) docs: refresh generator and helper URLs (#2852) Add standalone RSC upgrade guide for existing Pro apps (#2831) Raise docs version floor to 16.4.0 in install/demo guidance (#2610) # Conflicts: # CHANGELOG.md
justin808
added a commit
that referenced
this pull request
Mar 27, 2026
…olve-2835 * origin/main: (21 commits) docs: fix profiling node renderer command (#2863) generators: point Pro install fallback to upgrade guide (#2868) Add RSC Flight payload optimization guide (Article 7) (#2827) Migrate from deprecated Async::Variable to Async::Promise (#2832) docs: turn pro quick start into a gateway (#2862) Fix upload-assets endpoint duplicating bundles across directories (#2768) docs: fix stale docs links and help URLs (#2850) docs: replace dead pro.reactonrails.com links (#2851) docs: refresh generator and helper URLs (#2852) Add standalone RSC upgrade guide for existing Pro apps (#2831) Raise docs version floor to 16.4.0 in install/demo guidance (#2610) Fix release script: require changelog, fix RC version computation (#2848) Bump version to 16.5.0 Bump version to 16.5.0.rc.0 Update CHANGELOG.md for 16.5.0.rc.0 (#2847) Docs: add memory leak prevention guide for Node Renderer SSR (#2845) Docs: fix RSC migration gaps found during real-world migration (#2842) Add common mistakes sections to RSC migration guides (#2826) fix: preserve runtime env vars across Bundler.with_unbundled_env (#2836) Skip Pro CI workflows for Dependabot PRs (#2825) ... # Conflicts: # CHANGELOG.md
2 tasks
justin808
added a commit
that referenced
this pull request
Mar 28, 2026
## Summary - Stamped `### [16.5.1] - 2026-03-27` with two Pro fixes (PRs #2872, #2768) - Removed the `### [16.5.0.rc.0]` section (was already released as 16.5.0 stable) - Consolidated the 16.5.0 section with full entries (previously just said "no changes from rc.0") - Updated diff links at bottom of file ## Skipped PRs (docs/internal only) #2856, #2860, #2857, #2859, #2864, #2870, #2863, #2868, #2827, #2862, #2850, #2851, #2852, #2831, #2610, #2848 ## Test plan - [ ] Verify CHANGELOG.md formatting and diff links are correct - [ ] Run `rake release` (no args) after merge to publish 16.5.1 🤖 Generated with [Claude Code](https://claude.com/claude-code) <!-- CURSOR_SUMMARY --> --- > [!NOTE] > **Low Risk** > Low risk: changes are limited to `CHANGELOG.md` release notes and version/compare links with no runtime code modifications. > > **Overview** > Documents the `16.5.1` release by adding a new section under *Unreleased* with two **[Pro]** fixes (missing packaged rake tasks and avoiding duplicated bundles during remote renderer asset uploads). > > Cleans up the `16.5.0` entry by removing the `16.5.0.rc.0` section and updating the compare-link footer so `unreleased` now starts from `v16.5.1` and `16.5.0` compares from `v16.4.0`. > > <sup>Written by [Cursor Bugbot](https://cursor.com/dashboard?tab=bugbot) for commit 9a6b224. This will update automatically on new commits. Configure [here](https://cursor.com/dashboard?tab=bugbot).</sup> <!-- /CURSOR_SUMMARY --> <!-- This is an auto-generated comment: release notes by coderabbit.ai --> ## Summary by CodeRabbit * **Documentation** * Updated changelog to document version 16.5.1 release with consolidated version history and updated comparison links. <!-- end of auto-generated comment: release notes by coderabbit.ai --> Co-authored-by: Claude Opus 4.6 (1M context) <[email protected]>
justin808
added a commit
that referenced
this pull request
Mar 28, 2026
## Summary - Stamped `### [16.5.1] - 2026-03-27` with two Pro fixes (PRs #2872, #2768) - Removed the `### [16.5.0.rc.0]` section (was already released as 16.5.0 stable) - Consolidated the 16.5.0 section with full entries (previously just said "no changes from rc.0") - Updated diff links at bottom of file ## Skipped PRs (docs/internal only) #2856, #2860, #2857, #2859, #2864, #2870, #2863, #2868, #2827, #2862, #2850, #2851, #2852, #2831, #2610, #2848 ## Test plan - [ ] Verify CHANGELOG.md formatting and diff links are correct - [ ] Run `rake release` (no args) after merge to publish 16.5.1 🤖 Generated with [Claude Code](https://claude.com/claude-code) <!-- CURSOR_SUMMARY --> --- > [!NOTE] > **Low Risk** > Low risk: changes are limited to `CHANGELOG.md` release notes and version/compare links with no runtime code modifications. > > **Overview** > Documents the `16.5.1` release by adding a new section under *Unreleased* with two **[Pro]** fixes (missing packaged rake tasks and avoiding duplicated bundles during remote renderer asset uploads). > > Cleans up the `16.5.0` entry by removing the `16.5.0.rc.0` section and updating the compare-link footer so `unreleased` now starts from `v16.5.1` and `16.5.0` compares from `v16.4.0`. > > <sup>Written by [Cursor Bugbot](https://cursor.com/dashboard?tab=bugbot) for commit 9a6b224. This will update automatically on new commits. Configure [here](https://cursor.com/dashboard?tab=bugbot).</sup> <!-- /CURSOR_SUMMARY --> <!-- This is an auto-generated comment: release notes by coderabbit.ai --> ## Summary by CodeRabbit * **Documentation** * Updated changelog to document version 16.5.1 release with consolidated version history and updated comparison links. <!-- end of auto-generated comment: release notes by coderabbit.ai --> Co-authored-by: Claude Opus 4.6 (1M context) <[email protected]>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
https://pro.reactonrails.comlinks with the live canonical Pro docs landing page athttps://reactonrails.com/docs/pro/ReactOnRails::Utilsto stop sending users to the failing subdomainScope notes
react_on_rails/lib/generators/react_on_rails/pro_setup.rb, which is already tracked by Update Pro install CTA after reactonrails.com launch #2575/propage content /docs/pro/react-on-rails-pro.md, which is tracked by Docs: replace stale react-on-rails-pro overview references and retire broken page #2838Validation
Passed:
bundle exec rubocop react_on_rails/lib/react_on_rails/utils.rbgit diff --checkgit diff --name-only origin/main...HEAD | rg '\.md$' | xargs pnpm dlx [email protected] --checkcurl -I -Lchecks showing:https://reactonrails.com/docs/pro/->200https://pro.reactonrails.com->500Note
Low Risk
Low risk: changes are documentation link updates plus a single user-facing warning string, with no behavioral impact beyond directing users to a new URL.
Overview
Updates documentation to replace references to the dead
https://pro.reactonrails.comsite with the canonical Pro landing page athttps://reactonrails.com/docs/pro/across the README and multiple OSS docs.Also updates the
ReactOnRails::Utils.immediate_hydration_pro_install_warningmessage to point users at the new Pro docs URL whenimmediate_hydration: trueis used without the Pro gem.Written by Cursor Bugbot for commit 457f7d5. This will update automatically on new commits. Configure here.
Summary by CodeRabbit