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

[WIP] Upgrade Admin UI to Apollo Client 3 #3279

Closed
wants to merge 5 commits into from

Conversation

jordanoverbye
Copy link
Contributor

No description provided.

@request-info
Copy link

request-info bot commented Jul 23, 2020

It looks like we could use some more details. Good titles and descriptions help us fix things faster. If you can provide more information please update this issue.

@@ -35,7 +35,7 @@
"@keystonejs/oembed-adapters": "^5.1.4",
"apollo-cache-inmemory": "^1.6.6",
"apollo-client": "^2.6.10",
"apollo-upload-client": "^13.0.0",
"apollo-upload-client": "^14.1.0",
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Upgrading these versions of apollo-upload-client so that all versions in the monorepo are the same


import fetch from 'cross-fetch';

// This shouldn't be necessary, but it silences an annoying error from Apollo
// Client: https://github.com/apollographql/apollo-client/issues/3397#issuecomment-421433032
const fragmentMatcher = new IntrospectionFragmentMatcher({
Copy link
Contributor Author

@jordanoverbye jordanoverbye Jul 23, 2020

Choose a reason for hiding this comment

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

The IntrospectionFragmentMatcher has been removed in Apollo Client v3. See here apollographql/apollo-client#5073

@Vultraz
Copy link
Contributor

Vultraz commented Jul 23, 2020

I already have this done in #3125. Just needed to get one test working.

@jordanoverbye
Copy link
Contributor Author

Didn't see that @Vultraz. Going to close this.

@timleslie timleslie deleted the admin-ui-@apolloclient branch November 9, 2020 03:52
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.

2 participants