-
Notifications
You must be signed in to change notification settings - Fork 128
PR: ReactRouter v4 #45
Comments
Hi @Lutando It would be more than welcome. |
Hey @Lutando Do you think you'll have a chance to submit a PR for that? |
Hi @nicksp I will definitely do that on Saturday how does that sound? :) |
Also would you like it done in the old style (routes defined in a routes.js folder as it is now) or the new composable style as per their docs. I guess either way is a matter of preference. I am tend to lean towards the latter option. |
That's great @Lutando, whenever you have a time. For sure i'd expect latest recommended composable style :) |
Almost done with it just need to cleanup some things, expect a PR in the next few days |
@Lutando Cool, really excited about that ;) |
This is great! |
Would a PR with an update for ReactRouter v4 be welcomed?
The text was updated successfully, but these errors were encountered: