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
feat: Connection plugin features & fixes #674
Conversation
Codecov Report
@@ Coverage Diff @@
## develop #674 +/- ##
===========================================
+ Coverage 93.62% 93.63% +0.01%
===========================================
Files 44 44
Lines 2869 2877 +8
Branches 670 676 +6
===========================================
+ Hits 2686 2694 +8
Misses 181 181
Partials 2 2 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Suggestion:
Let's rebase the commit messages to be conventional commit compliant, then rebase-merge so that each change here shows up as a line item in the release notes
t2.field(key, val) | ||
t2.field(key, { | ||
...val, | ||
resolve: (fieldConfig as any).edgeFields?.[key] ?? defaultFieldResolver, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I fixed the pluginExtendConnection
but forgot that one indeed in #569 🤦♂️ (minus the defaultFieldResolver fallback 🙈)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks all good to me 👍 - Jason's suggestion would be great so that dripip
can properly format the release notes
Will skip requiring global extension fields
4b9f850
to
6220d85
Compare
Done. Merged as rebase into develop |
Fix: #450 custom edge fields are not being resolved
feat: #515 allow connection plugin cursor to be non-string type
feat: #399, add requireResolver: false to skip requiring global extension fields
feat: #479, allow promise to be returned from pageInfoFromNodes
feat: #456, export connection plugin args