-
Notifications
You must be signed in to change notification settings - Fork 218
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
feat(router): expose event.context.matchedRoute
#500
Conversation
Codecov Report
@@ Coverage Diff @@
## main #500 +/- ##
==========================================
+ Coverage 82.38% 82.74% +0.36%
==========================================
Files 31 31
Lines 3508 3518 +10
Branches 522 528 +6
==========================================
+ Hits 2890 2911 +21
+ Misses 618 607 -11
|
- also include `getRouterParam` function in readme
event.context.matchedRoute
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.
Thanks for pr and good explanations.
I have made some refactors to only expose it as event.context.matchedRotue?
preserving all route match information including handlers. I hope you like it!
For now using as an advanced/experimental feature since there might be some underlying changes.
π Linked issue
#499 499
β Type of change
π Description
Adds
event.context.matchedRoute
parameter which provides details of the route that the router matched on.π Checklist