Skip to content

WIP Update to React On Rails v6#287

Merged
justin808 merged 1 commit intomasterfrom
update-to-ror-6
May 20, 2016
Merged

WIP Update to React On Rails v6#287
justin808 merged 1 commit intomasterfrom
update-to-ror-6

Conversation

@justin808
Copy link
Copy Markdown
Member

@justin808 justin808 commented May 10, 2016

This PR is an example of updating from 5.2 to 6.0 of React on Rails.

This change is Reviewable

Comment thread lib/tasks/assets.rake
@@ -1,30 +0,0 @@
# lib/tasks/assets.rake
Copy link
Copy Markdown
Member Author

@justin808 justin808 May 11, 2016

Choose a reason for hiding this comment

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

UPGRADE NOTE: Note, we removed this WHOLE file!

Comment thread client/package.json
"build:dev:server": "webpack -w --config webpack.server.rails.build.config.js",
"build:server": "webpack --config webpack.server.rails.build.config.js",
"build:test": "npm run build:client && npm run build:server",
"build:production": "npm run build:production:client && npm run build:production:server",
Copy link
Copy Markdown
Member Author

@justin808 justin808 May 11, 2016

Choose a reason for hiding this comment

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

UPGRADE NOTE: Be sure to create 2 npm tasks like this. The first one, build:test is for building the assets for running tests. The second one build:production if for building assets before the precompile step. This MUST correspond to your config/initializers/react_on_rails.rb.

@justin808 justin808 force-pushed the update-to-ror-6 branch 2 times, most recently from 041bbee to f4935d1 Compare May 11, 2016 12:15
* update travis to ruby 2.3.1
@justin808 justin808 merged commit 5bbc68b into master May 20, 2016
"version": "3.0.4",
"from": "acorn@>=3.0.4 <4.0.0",
"resolved": "https://registry.npmjs.org/acorn/-/acorn-3.0.4.tgz"
"version": "3.1.0",
Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

UPGRADE NOTE: Do not worry about the diffs in this file. You will update this when you run npm i and maybe npm shrinkwrap. You will probably rm -rf node_modules. All those commands are to be run from the /client dir.

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.

1 participant