PR #71588 fix: uri encode dynamic routes in dev

fix: uri encode dynamic routes in dev #71588

samcx merged 12 commits into canary from sam/dynamic-routes-uri-encoding-dev

ijjk added created-by: Next.js team

ijjk added type: next

samcx requested a review from

ijjk 1 year ago

samcx requested a review from

wyattjoh 1 year ago

samcx marked this pull request as ready for review 1 year ago

samcx marked this pull request as draft 1 year ago

samcx force pushed from e843d673 to b1263646 1 year ago

ijjk commented on 2024-10-22

samcx force pushed from b1263646 to 3120aedd 1 year ago

samcx force pushed from 3120aedd to 9aba95ea 1 year ago

ijjk added tests

samcx marked this pull request as ready for review 1 year ago

samcx force pushed from cafc86ef to 7d8dc1ef 1 year ago

samcx requested a review from

ijjk 1 year ago

samcx force pushed from 7d8dc1ef to 589bdd03 1 year ago

samcx force pushed from 589bdd03 to d21a976e 1 year ago

wyattjoh approved these changes on 2024-10-29

samcx force pushed from 2006971c to 6e64bdb5 1 year ago

fix: uri encode dynamic routes in dev

5b934e1c

chore: move comment to private async

ab498972

chore: update comment in ResponseGenerator

1e68e4e1

test(app-dir): add e2e test for prerender encoding

a6d53dd7

test(prerender-encoding): remove unecessary root page

c9b7bb02

test(prerender-encoding): remove console.log

6371d9e5

refactor: move decodePathParams to a separate file/function

93c8528c

test(unit): add decode-path-params unit test

01d0a0e7

test(prerender-encoding): remove unecessary next.config

997c4e3b

test(prerender-encoding): switch to cheerio since its faster than nex…

241b8f8a

test: update import for decodePathParams

fd95d793

test: move unit test to same directory where function is

4b546d66

samcx force pushed from 6e64bdb5 to 4b546d66 1 year ago

samcx merged 35a3eca3 into canary 1 year ago

samcx deleted the sam/dynamic-routes-uri-encoding-dev branch 1 year ago

github-actions added locked

github-actions locked as resolved and limited conversation to collaborators 1 year ago

Reviewers

wyattjoh

ijjk

Assignees

No one assigned

Labels

type: next created-by: Next.js team locked tests

Milestone

No milestone

next.js fix: uri encode dynamic routes in dev #71588 Merged

fix: uri encode dynamic routes in dev #71588

next.js
fix: uri encode dynamic routes in dev
#71588

Merged