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

mention tutorial code in docs related to react-router integration #163

Merged
merged 3 commits into from
Dec 16, 2015
Merged

mention tutorial code in docs related to react-router integration #163

merged 3 commits into from
Dec 16, 2015

Conversation

marcellosachs
Copy link
Contributor

I had some trouble getting react-router to work, though after @justin808 pointed me to the tutorial code I was able to figure it out. Think it might be useful for others to see a mention of the tutorial code in the react-router docs.


For a fleshed out integration of react_on_rails with react-router, check out [React Webpack Rails Tutorial Code](https://github.com/shakacode/react-webpack-rails-tutorial), specifically the files:

[routes](https://github.com/shakacode/react-webpack-rails-tutorial/blob/master/client/app/bundles/comments/routes/routes.jsx)
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

let's use the full path /client/something for these

and let's use bullets for formatting

@marcellosachs
Copy link
Contributor Author

@justin808 added the full path to the file references and bullet formatting. Let me know if anything else should be adjusted...

@justin808
Copy link
Member

LGTM! Thanks!

justin808 added a commit that referenced this pull request Dec 16, 2015
mention tutorial code in docs related to react-router integration
@justin808 justin808 merged commit 009db0d into shakacode:master Dec 16, 2015
@marcellosachs
Copy link
Contributor Author

no problem! thanks for pointing me to the tutorial!

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