-
-
Notifications
You must be signed in to change notification settings - Fork 20
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
Comments
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… |
Experienced issues when updating to SS4.10. Reported them in |
This should be fixed now in #24 |
Lovely, thanks a lot. I've been using UncleCheese's fork until now, glad this is now merged/fixed. ❤️ |
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 duringdev/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#L14I'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.The text was updated successfully, but these errors were encountered: