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

Improve feature/apollo-pure #1720

Merged

Conversation

piglovesyou
Copy link
Collaborator

  • Upgrade dependencies, including
    • React 16.8
    • Apollo Client 2.5
    • Apollo, Babel, Webpack and more
  • Use new local state management of Apollo Client 2.5 (and removed apollo-link-state deps)
  • Apply more Flow Types (and removed prop-types deps)

rwhitworth and others added 13 commits February 28, 2019 23:48
* Upgrade all dependencies including Babel 7

1. ncu --upgradeAll
4. Leave "graphql" to be "^13.2.0" that apollo packages depend on
2. Apply patch of "npx babel-upgrade" output
3. Leave only necessary babel plugins
4. Pass "yarn fix" by disabling stricter rules

fixes kriasoft#1654 kriasoft#1626 kriasoft#1607

* Also ignore babel config file from webpack.config.js
* small README.md grammar change (kriasoft#1675)

* Upgrade all dependencies including Babel 7 (kriasoft#1673)

* Upgrade all dependencies including Babel 7
* Client Schema
* Simplify cache and cache defaults
* User in local state instead of context
mod: better suppressing flow error for .graphql import
fix 2 breaking changes:
* css-loader no longer accepts 'minimize' option
* isomorphic-style-loader uses new React Context now
@piglovesyou
Copy link
Collaborator Author

@langpavel Please consider merging this update. Thank you as always.

@langpavel langpavel merged commit 000a954 into kriasoft:feature/apollo-pure Mar 21, 2019
@langpavel
Copy link
Collaborator

langpavel commented Mar 21, 2019

@piglovesyou Squashed as always. Thanks

@piglovesyou
Copy link
Collaborator Author

Thank you!

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.

3 participants