-
Notifications
You must be signed in to change notification settings - Fork 156
feat: add composite type support for field resolvers #1341
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
Open
Noroth
wants to merge
93
commits into
master
Choose a base branch
from
ludwig/eng-8297-add-interface-support-for-field-resolvers
base: master
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Changes from 90 commits
Commits
Show all changes
93 commits
Select commit
Hold shift + click to select a range
51a306f
feat: implement field resolver logic in the planner
Noroth c92100a
Merge branch 'master' of github.com:wundergraph/graphql-go-tools into…
Noroth 49ab0e8
feat: update compiler and json builder to resolve fields
Noroth 4e6385d
chore: add mapping for resolve calls
Noroth 234bf5d
feat: handle list types
Noroth a84ccee
chore: update planner to handle optional types
Noroth 28ab022
chore: add tests for nullable and message types
Noroth ed61f46
chore: rename function
Noroth a0f6523
chore: rename directive
Noroth ade7d91
Merge branch 'master' of github.com:wundergraph/graphql-go-tools into…
Noroth 598aafe
feat: add field resolver support for entity calls
Noroth 33e58cb
chore: improvements
Noroth fb4c820
chore: lint
Noroth 1be8fab
Merge branch 'master' into ludwig/eng-6993-implement-field-resolver-f…
Noroth 46cde41
chore: improve compiler
Noroth 17a06a1
chore: fix nested list traversal
Noroth c7d5e7d
chore: add length check on type definitions
Noroth 3fa4321
chore: fix makefile
Noroth 4366b23
Merge branch 'master' into ludwig/eng-6993-implement-field-resolver-f…
Noroth eaea23f
Merge branch 'master' into ludwig/eng-6993-implement-field-resolver-f…
Noroth 5a04f78
chore: improve retrieving field selections from a selection set
Noroth 1ea6b55
Merge branch 'master' of github.com:wundergraph/graphql-go-tools into…
Noroth 9e74042
chore: add namespace to directive
Noroth 922dea8
Merge branch 'master' of github.com:wundergraph/graphql-go-tools into…
Noroth 0f0811c
Merge branch 'master' into ludwig/eng-6993-implement-field-resolver-f…
Noroth dc5b630
chore: update directive name
Noroth af8ea7c
chore: check for exact length
Noroth cf72a8d
chore: do not swallow error from marshalling
Noroth 329ef61
chore: add comments
Noroth 5021522
chore: improvements
Noroth 39bba4b
chore: remove unnecessary muxes
Noroth 9bae249
chore: use proper type instead of copying
Noroth 6755e2b
chore: reduce number of allocations
Noroth 218eb4c
chore: check error
Noroth 077bfd3
chore: rename file
Noroth ede645f
Merge branch 'master' into ludwig/eng-6993-implement-field-resolver-f…
Noroth 37db946
chore: improvements
Noroth 1ed5ae6
fix: remove unsafe operation report
Noroth 1aceadb
Merge branch 'master' into ludwig/eng-6993-implement-field-resolver-f…
Noroth a59091d
fix: traverse list
Noroth b6625fd
chore: improve memory allocation
Noroth c8acbda
Merge branch 'master' into ludwig/eng-6993-implement-field-resolver-f…
Noroth 1418287
chore: addressing pr comments
Noroth 59fdcbd
chore: improve variable names and function descriptions
Noroth 49f0dc4
chore: improve naming for directive
Noroth ca093fe
chore: move function to ast package
Noroth b5225e8
chore: improve naming
Noroth c9ebb59
chore: more polishing
Noroth 3eb8439
chore: add more comments
Noroth c178835
chore: use const
Noroth 555ad44
chore: resolve fields without required fields visitor
Noroth 503be72
feat: add support for composite types in the planner
Noroth ca0e63d
chore: rename field
Noroth 3eed050
chore: move length check
Noroth 57aaf86
Merge branch 'master' into ludwig/eng-6993-implement-field-resolver-f…
Noroth 1d07c41
Merge branch 'master' of github.com:wundergraph/graphql-go-tools into…
Noroth 7875e95
Merge branch 'ludwig/eng-6993-implement-field-resolver-for-graphql-op…
Noroth b72825d
Merge branch 'master' of github.com:wundergraph/graphql-go-tools into…
Noroth f6318c7
chore: rename type
Noroth 22b2b01
feat: implement handling for nested composite types
Noroth acae814
feat: support composite types in the federation visitor
Noroth 10490e6
chore: fixes incorrect parent mapping and adds tests for the datasource
Noroth 993c92b
chore: add tests for stack
Noroth 34f2b79
chore: remove println
Noroth fad062b
Merge branch 'master' into ludwig/eng-8297-add-interface-support-for-…
Noroth a165f7b
chore: fix incorrect variable name
Noroth 0d1466b
feat: handle nested types when using composite types in field resolvers
Noroth 33bbb70
chore: lint
Noroth cf88b88
Merge branch 'master' into ludwig/eng-8297-add-interface-support-for-…
Noroth 730de1a
chore: fix typo
Noroth 9395742
chore: improve naming
Noroth 8517bf3
chore: fix pagination
Noroth b77dba2
chore: fix imports
Noroth 87ea8e5
chore: improve handling for potential null fields
Noroth 6c0bd40
chore: remove exists
Noroth 331f49a
chore: better function name
Noroth 6ed7f6a
chore: update function comment
Noroth 8bd7847
Merge branch 'master' into ludwig/eng-8297-add-interface-support-for-…
Noroth d9d3692
chore: ignore nested field resolvers in composite types
Noroth 48dc314
chore: fix error message
Noroth cbab299
make test parallel
Noroth 2be4f06
chore: add comments
Noroth 6779a3f
chore: return error instead
Noroth 9171864
chore: simplify LeaveField logic
Noroth 08044c2
chore: improve comment
Noroth 881ca36
chore: skip node on duplicate field
Noroth 21c429f
Merge branch 'master' into ludwig/eng-8297-add-interface-support-for-…
Noroth 37df47a
fix: invalid index when skipping duplicate field
Noroth 4b04f32
chore: more improvements
Noroth ab44255
fix: incorrect error propagation
Noroth 805a7e1
fix: properly include introspection typename
Noroth f15e4c5
chore: check error
Noroth 231e13e
chore: field aliases are valid too
Noroth File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Uh oh!
There was an error while loading. Please reload this page.