Fix: redirects#916
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
💡 Enable Vercel Agent with $100 free credit for automated AI reviews |
| "/price-feeds/use-real-time-data/:path((?!pull-integration(?:/|$)).*)", | ||
| "/price-feeds/use-real-time-data/pull-integration/:path", |
There was a problem hiding this comment.
@aditya520 note that this has 2 cases:
http://localhost:3000/price-feeds/use-real-time-data redirects to http://localhost:3000/price-feeds/core/use-real-time-data
and
http://localhost:3000/price-feeds/use-real-time-data/something -> http://localhost:3000/price-feeds/use-real-time-data/pull-integration/something
I'm not sure if that's the intended behaviour
There was a problem hiding this comment.
The second one should be
http://localhost:3000/price-feeds/use-real-time-data/something -> http://localhost:3000/price-feeds/core/use-real-time-data/pull-integration/something
|
main issues are fixed, basically needs to be at the bottom of the redirects structure so nextjs can match others first |
537cc1c
into
chore(price-feeds)-combine-lazer-push-feeds
Description
Type of Change
Areas Affected
Checklist
pre-commit run --all-filesto check for linting errorsRelated Issues
Closes #
Additional Notes
Contributor Information
Screenshots