Skip to content
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

Add independent createRoutes method to support v4 react-router #82

Merged
merged 4 commits into from
Dec 14, 2018

Conversation

Joseph7451797
Copy link

Borrow createRoutes function from v3 react-router code base and make it as a internal function. This fixes the missing createRoutes method issue which caused by complete rewrite of v4 react-router.

kuflash and others added 3 commits September 23, 2018 11:20
@kuflash
Copy link
Owner

kuflash commented Nov 18, 2018

@Joseph7451797 very good work! Can you change destination branch to develop?

@Joseph7451797 Joseph7451797 changed the base branch from master to develop November 19, 2018 14:08
@Joseph7451797
Copy link
Author

@Joseph7451797 very good work! Can you change destination branch to develop?

@kuflash I already changed destination branch to develop, let me know if I need to do more actions or provide something.

@kuflash
Copy link
Owner

kuflash commented Dec 1, 2018

@Joseph7451797 I apologize for the late reply. Can you create example for use react-router-4 with react-router-sitemap. Maybe only code examples in README.md.

@Joseph7451797
Copy link
Author

@kuflash Code example of v4 react-router router setting is similar to old version one and is added to readme. Let me know if more detail is needed.

@kuflash
Copy link
Owner

kuflash commented Dec 14, 2018

@Joseph7451797 cool, that is all. Thank you!

@kuflash kuflash merged commit 5fdd6d0 into kuflash:develop Dec 14, 2018
@kuflash kuflash mentioned this pull request Jan 30, 2019
kuflash added a commit that referenced this pull request Jan 30, 2019
* Add support React Router v4 (#82)
* Release 1.2.0
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants