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

Update libraries and JSX for current versions 1/23/2015 #11

Merged
merged 4 commits into from
Feb 2, 2015

Conversation

justin808
Copy link
Member

  • Required a few JSX changes.
  • Don't use import for commonjs modules
  • key changed to eventKey for NavItem
  • Use different names for the the native input HTML elements compared to
    the react-bootstrap ones
  • Could not update the sass-loader without errors
  • Moved all Sass related parts of webpack to the webpack.hot.config.js
    as Rails version does not use the Webpack sass handlers.
  • Changed webpack.rails.config.js so that the loader for JSX does not
    use react-hot
  • Added the woff2 pattern for the file loader.
  • Changed from React.renderComponent to React.render in example.jsx.

@justin808
Copy link
Member Author

@mbreining @kimagure Any comments? I'll try to get the sass-loader updated next.

@justinwoo
Copy link

You can import CommonJS modules easily, just requires that you do import * as YOURNAME from 'module'

I admit it's not the best thing ever, but it'll work enough.

@mbreining
Copy link
Member

@justin808, the code in update-to-latest is deploying to Heroku fine but is not functional (none of the forms are showing up). The same issue exist on master. I'll have to look into it.

@mbreining
Copy link
Member

You can merge as far as I'm concerned.

* Required a few JSX changes.
* Don't use import for commonjs modules
* key changed to eventKey for NavItem
* Use different names for the the native input HTML elements compared to
  the react-bootstrap ones
* Could not update the sass-loader without errors
* Moved all Sass related parts of webpack to the webpack.hot.config.js
  as Rails version does not use the Webpack sass handlers.
* Changed webpack.rails.config.js so that the loader for JSX does not
  use react-hot
* Added the woff2 pattern for the file loader.
* Changed from React.renderComponent to React.render in example.jsx.
* New dependency uses proper array iteration
* Only affect the hot-reload version
@justin808
Copy link
Member Author

@mbreining I rebased master onto my branch and did a push -f.

@justin808 justin808 merged commit 17bcd21 into master Feb 2, 2015
@justin808 justin808 deleted the update-to-latest branch February 5, 2015 08:31
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.

None yet

3 participants