-
-
Notifications
You must be signed in to change notification settings - Fork 1.5k
docs(examples): clarify server.ts entrypoint updates for edge runtimes #6493
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: main
Are you sure you want to change the base?
docs(examples): clarify server.ts entrypoint updates for edge runtimes #6493
Conversation
📝 WalkthroughWalkthroughAdds documentation clarifications and instructions for configuring a custom server entry point and edge-runtime deployment (Cloudflare Workers) for the Paraglide React i18n example, plus README notes emphasizing that a custom Changes
Sequence Diagram(s)(omitted — changes are documentation-only and do not introduce new runtime control flow) Estimated code review effort🎯 2 (Simple) | ⏱️ ~10 minutes Suggested labels
Poem
🚥 Pre-merge checks | ✅ 3✅ Passed checks (3 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. 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.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Actionable comments posted: 1
🤖 Fix all issues with AI agents
In `@examples/react/start-i18n-paraglide/README.md`:
- Around line 91-96: The README shows a JSON code block for wrangler.toml which
is invalid; replace the JSON block with a TOML code block and use TOML comment
syntax, e.g., change the block labelled **Cloudflare Workers (`wrangler.toml`)**
to use ```toml``` and set the entry as main = "./src/server.ts" with comments
prefixed by # (not //) so the file is valid TOML for Wrangler.
76ed3b0 to
c8e3fdf
Compare
PR updates the Paraglide example documentation to clarify server entrypoint requirements for edge runtime deployments.
Changes:
server.tsneeds to be updated for edge runtimesparaglideMiddlewareprevents locale redirect bugs (e.g.,/de→/en)Issue was also mention here, but never properly documented. #6268
Summary by CodeRabbit
✏️ Tip: You can customize this high-level summary in your review settings.