-
-
Notifications
You must be signed in to change notification settings - Fork 1.6k
chore: Reorganize develop docs and change sidebar ordering #11619
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
Conversation
|
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
| @@ -1,5 +1,6 @@ | |||
| --- | |||
| title: Configuration | |||
| sidebar_order: 60 | |||
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.
I spaced most of the pages out by increments of 10 as it was easy to count and left holes for future docs to be added.
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.
Big THANKS for investing some time in improving our develop docs! I noticed that you haven't set up any redirects, could you add them for the pages you've moved?
You can add those to this file:
Line 4 in cbd9baa
| const developerDocsRedirects = [ |
| In terms of user experience, this feature plays a similar role as [Issue Grouping](/grouping/). | ||
| In terms of technical implementation, it is similar to [Data Scrubbing](/pii/). | ||
| In terms of user experience, this feature plays a similar role as [Issue Grouping](/backend/grouping/). | ||
| In terms of technical implementation, it is similar to [Data |
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.
Is this new line intentional?
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.
Nope, that was a mistake.
Updated! I didn't know there were redirects 😊 |
Bundle ReportChanges will decrease total bundle size by 348 bytes (-0.0%) ⬇️. This is within the configured threshold ✅ Detailed changes
|
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.
looks great! thanks for adding the redirects 🙏
DESCRIBE YOUR PR
The develop docs organization has grown organically and could benefit from some more cohesive organization. The new top level directories roughly map out to:
IS YOUR CHANGE URGENT?
Help us prioritize incoming PRs by letting us know when the change needs to go live.
SLA
Thanks in advance for your help!
PRE-MERGE CHECKLIST
Make sure you've checked the following before merging your changes: