-
Notifications
You must be signed in to change notification settings - Fork 2.6k
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(start/v2): fix links #7433
Conversation
|
docs/start/v2.md
Outdated
[meta-v2-rfc]: https://github.com/remix-run/remix/discussions/4462 | ||
[meta-v2-matches]: #the-matches-argument | ||
[v1-16-release-notes]: https://github.com/remix-run/remix/releases/tag/remix%401.16.0 | ||
[v2-dev-config]: ../other-api/dev-v2 | ||
[v2-dev-config]: ../other-api/dev | ||
[templates]: https://github.com/remix-run/remix/tree/main/templates | ||
[v2-dev-docs]: ../other-api/dev-v2 |
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 also wrong but I pushed a fix for it already
[remix-config]: ../file-conventions/remix-config | ||
[flat-routes]: https://github.com/remix-run/remix/discussions/4482 | ||
[meta-v2]: ../route/meta-v2 | ||
[meta-v2]: ../route/meta |
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 didn't get this one in my changes so if you merge latest dev into here I think this will still need to get merged 👍
8dd2f67
to
ae6804d
Compare
Couple of broken links in the upgrade docs, 2 right at the top 😬