Support for MethodFieldResolver with only DataFetchingEnvironment.
Maintainers usually reply within 1 day
Nobody has claimed this yet.
Assessment
- Difficulty
- 4/5
- Estimated time
- 3-5 days
- Newbie friendliness
- 25/100
Research direction
Start by locating the MethodFieldResolver entry point and reading how it currently matches resolver method parameters, especially DataFetchingEnvironment. Done means methods shaped like method (is|get|getField)?<Name>(*fieldArgs, DataFetchingEnvironment) are supported, while existing resolver forms continue to work.
Written by the indexing model from the issue text.
Description
- method
<name>(object, *fieldArgs [, DataFetchingEnvironment]) - method
is<Name>(object, *fieldArgs [, DataFetchingEnvironment]), only if the field returns a Boolean - method
get<Name>(object, *fieldArgs [, DataFetchingEnvironment]) - method
getField<Name>(object, *fieldArgs [, DataFetchingEnvironment]) - field
<name>
Currently only the above fieldResolvers are supported.
But there are use cases where are in using local context would be the only requirement.
Can methods with the following definition be supported?
method (is|get|getField)?<Name>(*fieldArgs, DataFetchingEnvironment) is supported.
The following hypothetical example shows that while loading imageBlob, we would want to know just the type of files supported. Since the parent of imageBlob is the image this data won't be available. Just the localContext is needed. In these cases, there is only need for DataFetchingEnvironment. So support for the above mentioned methodResolver would be cleaner.
type Folder {
supportedFileTypes: [String!]!
images: [Image!]!
}
type Image {
imageBlob: String
}
- Dominant language
- Kotlin
- Stars
- 824
- Forks
- 175
- Avg merge
- 2d 1h
- Merged PRs (30d)
- 59
Getting set up
This project ships no dev container, Dockerfile or contributing guide, so setting up is up to you: start from its README, and see our first-contribution guide for the general steps.
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
More from graphql-java-kickstart/graphql-java-tools
-
Difficulty 2/5 1-3 hours Newbie friendliness 85/100
graphql-java-kickstart/graphql-java-tools#893 ·
Maintainers usually reply within 1 day
-
enhancement
Difficulty 5/5 Over a week Newbie friendliness 35/100
graphql-java-kickstart/graphql-java-tools#905 ·
Maintainers usually reply within 1 day
-
Difficulty 3/5 1-2 days Newbie friendliness 48/100
graphql-java-kickstart/graphql-java-tools#900 ·
Maintainers usually reply within 1 day
-
Difficulty 3/5 1-2 days Newbie friendliness 72/100
graphql-java-kickstart/graphql-java-tools#889 ·
Maintainers usually reply within 1 day
-
Reject directives applied at the wrong location, without a required argument or twice in 15.0.0Openenhancement
Difficulty 3/5 1-2 days Newbie friendliness 72/100
graphql-java-kickstart/graphql-java-tools#884 ·
Maintainers usually reply within 1 day
All issues in graphql-java-kickstart/graphql-java-tools
Similar issues
-
bug
Difficulty 2/5 1-3 hours Newbie friendliness 78/100
software-mansion/enriched-markdown#1084 ·
Maintainers usually reply within 2 days
-
bug
Difficulty 2/5 1-3 hours Newbie friendliness 72/100
lysine-dev/okhttp#9814 ·
Maintainers usually reply within 1 day
-
Difficulty 2/5 1-3 hours Newbie friendliness 65/100
-
Difficulty 2/5 1-3 hours Newbie friendliness 72/100
getdictus/dictus-android#187 ·
Maintainers usually reply within 1 day
-
Difficulty 2/5 1-3 hours Newbie friendliness 79/100
bryangerlach/rdgen#319 ·