-
Notifications
You must be signed in to change notification settings - Fork 5.8k
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
Refactor Durable Objects WebSocket documentation #18344
Conversation
This PR changes current filenames or deletes current files. Make sure you have redirects set up to cover the following paths:
|
Deploying cloudflare-docs with
|
Latest commit: |
454a1ba
|
Status: | ✅ Deploy successful! |
Preview URL: | https://cc6cad3a.cloudflare-docs-7ou.pages.dev |
Branch Preview URL: | https://joshthoward-do-websocket-ref.cloudflare-docs-7ou.pages.dev |
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.
Leaving some suggestions to align with the style guide.
This PR deletes two WebSocket related files:
Where do we want to redirect each of these paths? We’ve only introduced one new file:
|
Co-authored-by: Jun Lee <[email protected]>
Co-authored-by: Jun Lee <[email protected]>
* Fix typo * Update order and lint files * Refactor Durable Objects WebSocket documentation * Fixing bad Prettier automation in <code> blocks. * Removing <br> and using MDX syntax. * Update src/content/docs/durable-objects/api/alarms.mdx Co-authored-by: Jun Lee <[email protected]> * Apply suggestions from code review Co-authored-by: Jun Lee <[email protected]> * Fixing broken links to deleted chapters. Adding public redirect. * Fixing broken links part 2 --------- Co-authored-by: Jun Lee <[email protected]>
Summary
Consolidate information on WebSockets into best practices.
Screenshots (optional)
Documentation checklist