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

Fix config.server_bundle_js file in generator #295

Merged

Conversation

aaronvb
Copy link
Member

@aaronvb aaronvb commented Feb 25, 2016

  • Change react_on_rails.rb to a thor template
  • Added server_rendering conditional
  • Added comment
  • Added tests

Fixes #294

Review on Reviewable

- Change react_on_rails.rb to a thor template
- Added server_rendering conditional
- Added comment
- Added tests
@justin808
Copy link
Member

Outstanding! @aaronvb Thanks! We need a list of React on Rails Heroes! You're on the list!

@justin808
Copy link
Member

Reviewed 4 of 4 files at r1.
Review status: all files reviewed at latest revision, all discussions resolved, some commit checks failed.


Comments from the review on Reviewable.io

justin808 added a commit that referenced this pull request Feb 25, 2016
Fix config.server_bundle_js file in generator
@justin808 justin808 merged commit ef51035 into shakacode:master Feb 25, 2016
@aaronvb aaronvb deleted the templatize-react-on-rails-config branch February 25, 2016 19:35
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