Skip to content

Fix config.server_bundle_js file in generator#295

Merged
justin808 merged 1 commit intoshakacode:masterfrom
aaronvb:templatize-react-on-rails-config
Feb 25, 2016
Merged

Fix config.server_bundle_js file in generator#295
justin808 merged 1 commit intoshakacode:masterfrom
aaronvb:templatize-react-on-rails-config

Conversation

@aaronvb
Copy link
Copy Markdown
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
Copy Markdown
Member

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

@justin808
Copy link
Copy Markdown
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
AbanoubGhadban pushed a commit that referenced this pull request Sep 25, 2025
* upgrade release-it
* honeybadger => @honeybadger-io/js
* upgrade express-busboy
* update rails & puma
* have CI run bundle update if bundle install fails
* fix CI job prereqs
* remove mini_racer
* update spec/dummy gemfile.lock
AbanoubGhadban pushed a commit that referenced this pull request Sep 26, 2025
* upgrade release-it
* honeybadger => @honeybadger-io/js
* upgrade express-busboy
* update rails & puma
* have CI run bundle update if bundle install fails
* fix CI job prereqs
* remove mini_racer
* update spec/dummy gemfile.lock
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