-
Notifications
You must be signed in to change notification settings - Fork 41
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(packages/tuono-router): add replace
method to router and replace
prop to Link
#596
Conversation
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.
Left two comments about two minor adjustments,
everything else looks goods to me.
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.
Left few minor comments. LGTM
Thanks
@m-jovan, sorry to bother you, could you please re-request a review after you have completed the changes (more info)? |
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.
🔥
replace
method to router and replace
prop to Link
Checklist
Related issue
Fixes #537
Overview
This PR does the following:
replace
method to the instance returned byuseRouter
hookuseRouter
types to avoid repetition with addingreplace
methodreplace
prop toLink
component that replaces the state by calling router's now addedreplace
methodRelated docs will be added with tuono-labs/tuono-documentation#30