Use clientState cache if provided in apollo-boost

[petetnt]

This PR fixes #3220 by using cache supplied to clientState config if passed instead of the default InMemoryCache.

(index.ts lines 62-80 were formatted by prettier and contain no new changes).

Signed-off-by: petetnt pete.a.nykanen@gmail.com

Checklist:

• If this PR is a new feature, please reference an issue where a consensus about the design was reached (not necessary for small changes)
• Make sure all of the significant new logic is covered by tests

[hwillson]

Thanks for submitting this PR @petetnt! Unfortunately, this isn't something we're ready to merge at this point. I've outlined the reasons why in #3220 (comment), but just to re-cap:

apollo-boost is intended to be an opinionated approach for working with Apollo Client. The fact that the default InMemoryCache instance can't be changed, is by design. Anyone who wants to use a different cache implementation is advised to setup an ApolloClient instance manually.

I've left that issue open for discussion though, so if there are compelling reasons why this should be a feature, we'll definitely re-consider a variation of this PR. Thanks again for your time and contribution!

[petetnt]

Sent a message to the original issue.

[hwillson]

This functionality has been added via #3561, but I just wanted to say thanks again for submitting this PR originally @petetnt. Your PR here (and your discussion in #3220) helped get this feature in.

[petetnt]

Thanks!