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.

Sign up for GitHub

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

Jump to bottom

Fix NextUrl trailing slash normalize for data route #41311

Merged

ijjk merged 1 commit into vercel:canary from ijjk:fix/next-url-trailing-slash

Oct 10, 2022

Member

ijjk commented Oct 10, 2022 •

edited

x-ref: slack thread

Bug

Related issues linked using fixes #number
Integration tests added
Errors have a helpful link attached, see contributing.md


          Fix NextUrl trailing slash normalize for data route

ijjk added created-by: Next.js team type: next labels

ijjk marked this pull request as ready for review

October 10, 2022 23:50

ijjk requested review from timneutkens, shuding and huozhi as code owners

October 10, 2022 23:50

ijjk merged commit acb46e1 into vercel:canary

ijjk deleted the fix/next-url-trailing-slash branch

October 10, 2022 23:50

styfle added the hacktoberfest-accepted label

Kikobeats pushed a commit to Kikobeats/next.js that referenced this pull request


          Fix NextUrl trailing slash normalize for data route (vercel#41311)

1c3e42a

x-ref: [slack
thread](https://vercel.slack.com/archives/C045FKE5P51/p1665074590321179)

## Bug

- [x] Related issues linked using `fixes #number`
- [x] Integration tests added
- [ ] Errors have a helpful link attached, see `contributing.md`

github-actions bot locked as resolved and limited conversation to collaborators

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.