-
Notifications
You must be signed in to change notification settings - Fork 28.9k
wip: root params in route handlers #80684
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
base: lubieowoce/root-params-types
Are you sure you want to change the base?
wip: root params in route handlers #80684
Conversation
Warning This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
This stack of pull requests is managed by Graphite. Learn more about stacking. |
f70791b
to
23b8248
Compare
CodSpeed Performance ReportMerging #80684 will not alter performanceComparing Summary
|
23b8248
to
87f9b63
Compare
41da255
to
1674139
Compare
87f9b63
to
bf41c6b
Compare
1674139
to
b129141
Compare
bf41c6b
to
f692cde
Compare
68666ef
to
8a43e92
Compare
f692cde
to
bf12851
Compare
8a43e92
to
34ecb17
Compare
7fc10cb
to
6248ef5
Compare
34ecb17
to
7b579f8
Compare
6248ef5
to
32e74d6
Compare
752d7e6
to
6cf65cb
Compare
32e74d6
to
2a607fc
Compare
2a607fc
to
b4fe9d1
Compare
6cf65cb
to
6b6d73a
Compare
@@ -274,6 +274,7 @@ export async function adapter( | |||
const requestStore = createRequestStoreForAPI( | |||
request, | |||
request.nextUrl, | |||
{}, // TODO(root-params): compute and pass real rootParams |
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.
TODO: edge
519d35e
to
17ac63c
Compare
6b6d73a
to
11ede39
Compare
17ac63c
to
bfe1fe3
Compare
11ede39
to
80bdbc2
Compare
bfe1fe3
to
d93fb49
Compare
.../app-dir/app-root-params-getters/fixtures/simple/app/[lang]/[locale]/route-handler/route.tsx
Show resolved
Hide resolved
80bdbc2
to
639d677
Compare
ae507a7
to
e9ce96d
Compare
cb48272
to
e67bd13
Compare
e9ce96d
to
2caaeca
Compare
e67bd13
to
c220c1f
Compare
2caaeca
to
540330c
Compare
This PR enables the use of root params in route handlers by: 1. Modifying the app route module to extract root params from context.params based on rootParamNames 2. Passing these root params to the request store in route handlers 3. Removing the error that previously prevented using root params in route handlers 4. Simplifying the rootParams handling by directly using string arrays instead of ParamInfo objects 5. Replacing the custom encodeToBase64 utility with standard Buffer.from().toString('base64')
540330c
to
85295bd
Compare
c220c1f
to
b0cdc2d
Compare
No description provided.