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
chore: graphql v5 schema changes #19685
Conversation
508ce01
to
7a1467d
Compare
0d5ebf3
to
321306f
Compare
Converting to draft I'm gonna try sth even better |
321306f
to
c8a093b
Compare
c8a093b
to
279eb44
Compare
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.
It's taking shape! Can't wait to have the new format
It seems that e2e are failing
packages/plugins/graphql/server/src/services/builders/queries/single-type.ts
Outdated
Show resolved
Hide resolved
packages/plugins/graphql/server/src/services/builders/mutations/single-type.ts
Show resolved
Hide resolved
packages/plugins/graphql/server/src/services/builders/mutations/collection-type.ts
Outdated
Show resolved
Hide resolved
packages/plugins/graphql/server/src/services/builders/filters/content-type.ts
Show resolved
Hide resolved
packages/plugins/graphql/server/src/services/builders/mutations/collection-type.ts
Show resolved
Hide resolved
2b13167
to
0988cf7
Compare
Size Change: +75 B (0%) Total Size: 2.46 MB
ℹ️ View Unchanged
|
0988cf7
to
92e54aa
Compare
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.
LGTM
What does it do?
Migrates the graphql plugin to v5.
query_connection
createXLocalization
in favor of the documents serviceWill do after D&P disabling
Why is it needed?
How to test it?