-
-
Notifications
You must be signed in to change notification settings - Fork 8.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
docs: update Cloudflare Pages deployment link #10901
Conversation
✅ [V2]Built without sensitive environment variables
To edit notification comments on pull requests, go to your Netlify site configuration. |
⚡️ Lighthouse report for the deploy preview of this PR
|
… Pages deployment link
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.
Thanks 👍
Yes this change is appropriate, I prefer to link to the official docs if it exists :)
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.
Thanks 👍
Yes this change is appropriate, I prefer to link to the official docs if it exists :)
Pre-flight checklist
Motivation
The Deployment guide currently links to a dev.to blog post for deploying to Cloudflare Pages. However, there is now an official Cloudflare Pages documentation specifically for deploying Docusaurus. I was able to set up my deployment by following this official documentation😀 Therefore, I have updated the link to point to the official documentation instead. Is this change appropriate?
Test Plan
Test links
👇️updated and works good :)
Deploy preview: https://deploy-preview-10901--docusaurus-2.netlify.app/docs/deployment/#deploying-to-cloudflare-pages
Related issues/PRs
N/A
Thank you!