Grammar fixes for self host - #2848
Conversation
📝 WalkthroughWalkthroughUpdated self-hosting documentation for hosting selection, configuration, AWS deployment, and migration procedures. Changes clarify wording, present-tense descriptions, punctuation, formatting, and private DNS instructions without changing APIs or procedures. ChangesSelf-hosting documentation
Estimated code review effort: 1 (Trivial) | ~5 minutes Possibly related PRs
Suggested reviewers: 🚥 Pre-merge checks | ✅ 4 | ❌ 1❌ Failed checks (1 inconclusive)
✅ Passed checks (4 passed)
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 |
There was a problem hiding this comment.
Actionable comments posted: 2
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Inline comments:
In `@client/www/app/docs/self-hosting/page.md`:
- Around line 19-21: Update the self-hosting setup instructions to make
“migrating from Instant Cloud” an explicit hosting choice alongside side
projects and startups. Direct migration requests to the Migrate guide, while
preserving the existing cost guidance and VPS/AWS selection for the other
choices.
- Around line 132-134: Update the create-instant-app documentation to state that
both INSTANT_CLI_API_URI and INSTANT_CLI_DASH_URI must be provided for the
self-hosted dashboard URL to be added to instant.config.ts, while preserving the
existing authentication context.
🪄 Autofix
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: Repository UI
Review profile: CHILL
Plan: Pro Plus
Run ID: 20582988-79cd-484d-8802-0069bf38d3f9
📒 Files selected for processing (3)
client/www/app/docs/self-hosting/aws/page.mdclient/www/app/docs/self-hosting/migrate/page.mdclient/www/app/docs/self-hosting/page.md
|
View Vercel preview at instant-www-js-sm-self-host-grammar-jsv.vercel.app. |
What it says on the tin!