search

adamsoffer / next-apollo

React higher-order component for integrating Apollo Client with Next.js
MIT License
481 stars 64 forks source link

Added Warning about Cache and SSR #12

Closed couturecraigj closed 6 years ago

couturecraigj commented 6 years ago

After encountering this issue it makes sense to provide this warning.

couturecraigj commented 6 years ago

https://github.com/ads1018/next-apollo/blob/5d7bdf3650e7d2a0df9fe291ccf2ef58ccbe10b9/src/initApollo.js#L13-L20

There this an issue with cache, if a user passes in their own in here. It may make sense to both give this warning and drop the cache from apolloConfig to prevent any further issues.

adamsoffer commented 6 years ago

Thanks for the pr!