Skip to content
This repository has been archived by the owner on Nov 6, 2020. It is now read-only.

Update JS dependencies #3710

Merged
merged 12 commits into from
Dec 5, 2016
Merged

Update JS dependencies #3710

merged 12 commits into from
Dec 5, 2016

Conversation

ngotchac
Copy link
Contributor

@ngotchac ngotchac commented Dec 2, 2016

This will update every JS dependencies to their latest version, except 2.
Some unused dependencies have been removed too.

Not updated:

@ngotchac ngotchac added A0-pleasereview 🤓 Pull request needs code review. M7-ui labels Dec 2, 2016
@coveralls
Copy link

Coverage Status

Changes Unknown when pulling 640022c on ng-ui-updates into ** on master**.

@@ -98,7 +95,7 @@ if (module.hot) {
// // <AppContainer>
// // <ContextProvider api={ api } muiTheme={ muiTheme } store={ newStore }>
// // <MainApplication
// // routerHistory={ routerHistory }
// // routerHistory={ hashHistory }
Copy link
Contributor

Choose a reason for hiding this comment

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

Can we remove this block now?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

We sure can. It's just that right now whenever the Redux store is modified, the whole app has to refresh. With a bit of work, this could be hot reloaded

@jacogr jacogr added A8-looksgood 🦄 Pull request is reviewed well. and removed A0-pleasereview 🤓 Pull request needs code review. labels Dec 5, 2016
@jacogr jacogr merged commit 9a43e6d into master Dec 5, 2016
@jacogr jacogr deleted the ng-ui-updates branch December 5, 2016 10:48
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
A8-looksgood 🦄 Pull request is reviewed well.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants