-
Notifications
You must be signed in to change notification settings - Fork 2.1k
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
resolves parameters in headers location #2095
resolves parameters in headers location #2095
Conversation
@TooTallNate not sure if it's ok to bump the canary version number in package.json. I was using it to see if my local |
This reverts commit 5857489.
Co-Authored-By: williamli <email@williamli.dev>
Codecov Report
@@ Coverage Diff @@
## canary #2095 +/- ##
=========================================
+ Coverage 4.5% 4.56% +0.06%
=========================================
Files 243 243
Lines 8575 8582 +7
Branches 876 878 +2
=========================================
+ Hits 386 392 +6
Misses 8172 8172
- Partials 17 18 +1
Continue to review full report at Codecov.
|
|
@TooTallNate https://developer.mozilla.org/en-US/docs/Web/HTTP/Headers |
…-parameters' into feature/now-dev-headers-location-parameters
@TooTallNate done |
This is a follow up to #2095 to move the `resolveRouteParameters()` funcrtion to the top-level and other slight optimizations like not re-parsing the URL multiple times.
* [now dev] Clean up `dev-router` a bit This is a follow up to #2095 to move the `resolveRouteParameters()` funcrtion to the top-level and other slight optimizations like not re-parsing the URL multiple times. * Remove unused `qs` import
* version bump (canary) * inject paramters into headers.Location fixes #2092 * refactored route parameter resolving function * Revert "version bump (canary)" This reverts commit 5857489. * Update src/commands/dev/lib/dev-router.ts Co-Authored-By: williamli <email@williamli.dev> * refactor headers resolution * try to resolve all parameters inside each child of headers
* [now dev] Clean up `dev-router` a bit This is a follow up to #2095 to move the `resolveRouteParameters()` funcrtion to the top-level and other slight optimizations like not re-parsing the URL multiple times. * Remove unused `qs` import
No description provided.