-
Notifications
You must be signed in to change notification settings - Fork 92
QueryString handling #1
Comments
This is an interesting suggestion! I would not use it for the routing itself tough, but make the query/GET params available in the result function. For example, if you have this URL: const routes = {
'/section/:sectionName': ({sectionName}, {order}) => <RenderSection name={sectionName} sortOrder={order} />
}; Having a second argument passed to the result function would not break the existing API. |
This is the exact question I was going to ask and the exact answer I was hoping to hear. Hope you’ll consider adding this. This project looks awesome! Edit: just realized you tagged this as “Done(to be released)” yay!! |
Released with Can be used with the |
* fix the `wth` exception * add comment for the wth error * fix indentation
Merge in UI/hookrouter from patched to master * commit '902a1bd11ef3f1798f130fbfa7425784ff53cb29': add history state 1.5.0 1.4.0 WTH exception handling (#1) fix package lock bump version number update package json fix some vulnerabilities update package name and add README entry Revert "add dist files so we can directly install this package from the sources" fix build add build script add dist files so we can directly install this package from the sources add to gitignore fix parantheses Remove extra closing parentheses Update Link.js Update Link.js Support React 17
Nice and clean. Do you plan to have a way of handing query string?
The text was updated successfully, but these errors were encountered: