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

Breaking: Upgrade to MUI v1 stable release (fixes #38) #39

Merged
merged 1 commit into from
May 21, 2018
Merged

Conversation

Giners
Copy link
Owner

@Giners Giners commented May 21, 2018

This commit upgrades to the first stable release of MUI (yay!). This commit accounts for breaking changes made in v1.0.0 of MUI. We mainly had to account for how MUI components were imported now as well as upgrade to React 16.3 as it is a requirement for MUI v1.0.0 now. As a result we are also declaring this commit to be a breaking change. This is to account for the fact that the peer dependencies have changed for MUI as well as what versions of the peer dependencies are required for
MUI and React.

Thanks to @charliedotau and @charlax on the heads up about the new version of MUI and the breaking changes we need to account for.

This commit upgrades to the first stable release of MUI (yay!). This
commit accounts for breaking changes made in v1.0.0 of MUI. We mainly
had to account for how MUI components were imported now as well as
upgrade to React 16.3 as it is a requirement for MUI v1.0.0 now. As a
result we are also declaring this commit to be a breaking change. This
is to account for the fact that the peer dependencies have changed for
MUI as well as what versions of the peer dependencies are required for
MUI and React.

Thanks to @charliedotau and @charlax on the heads up about the new
version of MUI and the breaking changes we need to account for.
@Giners Giners self-assigned this May 21, 2018
@Giners Giners requested review from mattbailey and bheftel May 21, 2018 13:33
@Giners Giners merged commit 2f0137f into master May 21, 2018
@Giners Giners deleted the fix/mui-v1 branch May 21, 2018 16:22
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

2 participants