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

GraphQL classes not compatible with upcoming GraphQL 4 module #23

Closed
bummzack opened this issue Jun 28, 2021 · 4 comments
Closed

GraphQL classes not compatible with upcoming GraphQL 4 module #23

bummzack opened this issue Jun 28, 2021 · 4 comments

Comments

@bummzack
Copy link

The new GraphQL 4 module removed the need for some of the classes, such as QueryCreator. Therefore the installation of this module next to GraphQL 4 will result in fatal errors during dev/build

Maybe the issue could be solved similarly to the asset-admin module, where they moved these classes into a _legacy folder: https://github.com/silverstripe/silverstripe-asset-admin/tree/1/_legacy/GraphQL and prevented class loading like this: https://github.com/silverstripe/silverstripe-asset-admin/blob/1/_legacy/GraphQL/ReadFileUsageQueryCreator.php#L14

I'm unsure about the ScaffoldingProvider (https://github.com/gorriecoe/silverstripe-link/blob/master/src/models/Link.php#L32) though, as it is no longer available in the latest version.

@bummzack
Copy link
Author

bummzack commented Jun 28, 2021

The upgrade-path for scaffolders is described as follows: https://docs.silverstripe.org/en/4/developer_guides/graphql/upgrading/#scaffoldingproviders-are-now-schemaupdaters

Not sure if that can be accomplished within the same codebase though (eg. creating a module that is compatible with both GraphQL 3 and 4). Maybe @unclecheese knows…

@robertvanlienden
Copy link

Experienced issues when updating to SS4.10. Reported them in silverstripe-linkfield, see the issue below;
elliot-sawyer/silverstripe-linkfield#33

@elliot-sawyer
Copy link
Owner

This should be fixed now in #24

@bummzack
Copy link
Author

Lovely, thanks a lot. I've been using UncleCheese's fork until now, glad this is now merged/fixed. ❤️

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

No branches or pull requests

3 participants