Duke-MatSci / materialsmine

MaterialsMine App
6 stars 1 forks source link

Apollo Client cache and link #277

Open aswallace opened 1 year ago

aswallace commented 1 year ago

Leaving cache out of new ApolloClient results in the following error:

In order to initialize Apollo Client, you must specify 'link' and 'cache' properties in the options object.
These options are part of the upgrade requirements when migrating from Apollo Client 1.x to Apollo Client 2.x.
For more information, please visit: https://www.apollographql.com/docs/tutorial/client.html#apollo-client-setup

We will need to look into whether to address this or leave as is.

_Originally posted by @aswallace in https://github.com/Duke-MatSci/materialsmine/pull/276#discussion_r1026894439_

tholulomo commented 1 year ago

@aswallace I noticed the issue is marked as merged. Does that mean this issue is now resolved cos I don't think so.

aswallace commented 1 year ago

@tholulomo No, the branch was merged with link and cache, but the issue itself wasn't resolved. We still needed to decide whether we want to leave link and cache in the Apollo initialization