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: update MP GraphQL to v15 and peer dependencies [DRAFT / IN-PROGRESS] #5455
base: main
Are you sure you want to change the base?
Conversation
@ardatan/aggregate-errorAuthor: Sindre Sorhus Description: Create an error from multiple errors Homepage: https://github.com/sindresorhus/aggregate-error#readme
|
I will remove reviewers until I fix the broken tests |
(cherry picked from commit 34f3b527fd731ceabeb59b53a3b3610d67b62679)
(cherry picked from commit 9194bde119ad00fc1762c66e59db9ca578f7e9cc)
This reverts commit b6eb6ed7cb6e73707694faa21278cc5708fec016.
This reverts commit 2333c383ad81b08e99e00f1ff74c17c4ec0dc853.
7c489de
to
b78dc99
Compare
This PR updates MP GraphQL to v15 and all required dependencies to the first version that supports that.
As I started upgrading packages, I had some troubles with enums and had to reapply patches we made. Eventually, I was happy to see that the
graphql-tools
v15 does the same as we do in our patch and there was no need for the patch anymore.How I QA'd my changes:
Further QA
Any volunteers to QA this together if it gets merged to staging
What comes next after this?
graphql
15 andexpress-graphql
both have experimental@defer
andstream
branches. I am planning on evaluating the effort for such an integration later