Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Provider for creating missing resolver data fetcher to provide options and field definition to factory #742

Conversation

timward60
Copy link
Contributor

Provider for creating missing resolver data fetcher to provide options and field definition to factory

Resolves #741

Checklist

  • Pull requests follows the contribution guide
  • New or modified functionality is covered by tests

Description

Allow missing field resolver data fetcher usage a hook to the field definition and options. Moves away from provide a concrete data fetcher implementation and provide a functional interface (aka Provider) when creating the missing resolver data fetchers and pass the field definition and schema parser options for context.

Copy link
Collaborator

@oryan-block oryan-block left a comment

Choose a reason for hiding this comment

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

Looks good I just have some formatting nitpicks

Copy link
Collaborator

@oryan-block oryan-block left a comment

Choose a reason for hiding this comment

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

.

@timward60
Copy link
Contributor Author

Thanks @oryan-block for the review. Applied your suggestions. Let me know if there is anything else!

@timward60 timward60 requested a review from oryan-block April 28, 2023 20:46
@oryan-block oryan-block merged commit 8ab25b6 into graphql-java-kickstart:master May 1, 2023
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Allow missing field resolver data fetcher usage a hook to the field definition and options
2 participants