-
-
Notifications
You must be signed in to change notification settings - Fork 208
Upgrade to @material-ui/core - 1.0.0-rc.1 #213
Conversation
@sulfurious Wow, this is a great first contribution to this project, thank you very much! 🎉 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please update React, then it's 💯
@@ -52,14 +53,13 @@ | |||
"enzyme-to-json": "^3.2.2", | |||
"gh-pages": "^1.0.0", | |||
"jest": "^21.2.1", | |||
"material-ui": "^1.0.0-beta.30", | |||
"react": "^16.1.1", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
React (and ReactDOM) should be updated to ^16.3.0
, since Material-UI requires at least 16.3.0 as of rc.0
"react": "^16.1.1", | ||
"react-dom": "^16.1.1", | ||
"react-test-renderer": "^16.2.0", | ||
"standard": "^10.0.2" | ||
}, | ||
"peerDependencies": { | ||
"material-ui": "^1.0.0-beta.30", | ||
"@material-ui/core": "^1.0.0-rc.1", | ||
"react": "^16.0.0", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
See above.
import { green } from 'material-ui/colors' | ||
import Avatar from '@material-ui/core/Avatar' | ||
import Chip from '@material-ui/core/Chip' | ||
// import AutoComplete from '@material-ui/core/AutoComplete' |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think we can remove this line. AutoComplete
will never be in @material-ui/core
😢
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hey @leMaik I'll take care of the AutoComplete references and the React 16.3 requirement today
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@leMaik nm - you already took care of that :-)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Indeed, I just couldn't wait. 😄
Upgrade to material-ui-next rc
https://github.com/mui-org/material-ui/releases/tag/v1.0.0-rc.1