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

Compatibility with wp-graphql v0.8.0 #15

Closed
hrosb opened this issue Mar 26, 2020 · 3 comments
Closed

Compatibility with wp-graphql v0.8.0 #15

hrosb opened this issue Mar 26, 2020 · 3 comments

Comments

@hrosb
Copy link

hrosb commented Mar 26, 2020

Hi,

Thank you for your hard work with this plugin. I hope all is well with you guys during these special days.

Seems like v0.8.0 of wp-graphql introduced some breaking changes.
Details:
https://github.com/wp-graphql/wp-graphql/releases/tag/v0.8.0

Specifically the changes to the AbstractConnectionResolver is breaking querying for fields inside a form.

Probably #14 is releated to this. My error was very similar:
Fatal error: Class WPGraphQLGravityForms\Connections\FormFieldConnectionResolver contains 4 abstract methods and must therefore be declared abstract or implement the remaining methods (WPGraphQL\Data\Connection\AbstractConnectionResolver::get_loader_name, WPGraphQL\Data\Connection\AbstractConnectionResolver::get_ids, WPGraphQL\Data\Connection\AbstractConnectionResolver::is_valid_offset, ...) in (path)/wp-graphql-gravity-forms/src/Connections/FormFieldConnectionResolver.php on line 9

Any work being done to update the plugin to work with the new version?

@Twansparant
Copy link

Jup, I'm running into the same issues.
Rolling back to wp-graphql v0.7.1 fixed these errors.

@kellenmace
Copy link
Contributor

Hi @hrosb & @Twansparant. Yes, the plugin does need to be updated to work with the new abstract connection resolver / deferred resolution model that WPGraphQL has introduced. I plan to push an update soon so that this plugin is once again compatible with the latest version of WPGrapQL.

@kellenmace
Copy link
Contributor

@hrosb & @Twansparant: FYI – have updated the plugin to work with the latest version of WPGraphQL (0.12.3).

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