-
Notifications
You must be signed in to change notification settings - Fork 10.3k
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
Update Routing docs main page #19700
Conversation
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.
This is awesome @laurieontech! It spells out a lot of the magic of routing really clearly.
I left a few comments for your consideration 🙂
Co-Authored-By: Sidhartha Chatterjee <[email protected]>
Co-Authored-By: Kyle Gill <[email protected]>
Co-Authored-By: Kyle Gill <[email protected]>
Co-Authored-By: Kyle Gill <[email protected]>
Awesome, thanks @gillkyle! Made some adjustments to your edits, but incorporated them all. |
… the docs and point to the content inside the main routing landing page.
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 think this looks great 👍
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.
my suggestion
Co-Authored-By: Michael <[email protected]>
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.
This is going to be so helpful for readers! I left a few suggestions for links and requests for examples...let me know what you think.
Co-Authored-By: Marcy Sutton <[email protected]>
Co-Authored-By: Marcy Sutton <[email protected]>
Co-Authored-By: Marcy Sutton <[email protected]>
Co-Authored-By: Marcy Sutton <[email protected]>
Co-Authored-By: Marcy Sutton <[email protected]>
Co-Authored-By: Marcy Sutton <[email protected]>
Co-Authored-By: Marcy Sutton <[email protected]>
Co-Authored-By: Marcy Sutton <[email protected]>
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.
Great job on this, @laurieontech! Approving so we can merge it in. 👍
Merging this, as I believe it's ready! |
Description
This PR is part of addressing the 25 Learning Workflows and is part of the "Building Apps with Gatsby" improvements.
Related Issues
Addresses #18993