wip: root params in route handlers#80684
Draft
lubieowoce wants to merge 3 commits into
Draft
Conversation
Merged
Member
Author
|
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
Footnotes |
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
ae507a7 to
e9ce96d
Compare
cb48272 to
e67bd13
Compare
e9ce96d to
2caaeca
Compare
e67bd13 to
c220c1f
Compare
540330c to
85295bd
Compare
c220c1f to
b0cdc2d
Compare
85295bd to
e75141d
Compare
249d194 to
31acff9
Compare
fe0b582 to
1d2048b
Compare
31acff9 to
43c80f4
Compare
1d2048b to
d5a5e4f
Compare
43c80f4 to
a43f796
Compare
d5a5e4f to
e6cf252
Compare
074d63e to
e66a8ca
Compare
bd975d4 to
b99ad3b
Compare
e66a8ca to
3258420
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')
3258420 to
fe069bb
Compare
b99ad3b to
c0d7e49
Compare
Contributor
|
@lubieowoce do you have an update on when root-params in route-handlers will be supported? thanks! |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.

No description provided.