-
Notifications
You must be signed in to change notification settings - Fork 2.4k
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
Upgrade graphql to 7.7.2 #15129
Upgrade graphql to 7.7.2 #15129
Conversation
@hishamco I just merged your code from the main branch and fixed the compilation errors, but the unit tests haven't been fixed yet |
Here's a question. I plan to add logging functionality to my generation unit test project. |
I prefer to do it in a separate PR |
OK, here #15130 , I'm going to merge the PR into this branch |
There seems to be only one problem for now
|
@hishamco There are too many expired warnings, after fixing all the warnings, upgrade 7.7.2 will not report errors, but the problem is still in the type registration |
Because there're deprecated members |
I've actually fixed all deprecated members The problem now is
|
@sebastienros can we triage this today? |
...OrchardCore.Modules/OrchardCore.ContentFields/GraphQL/Fields/ObjectGraphTypeFieldProvider.cs
Outdated
Show resolved
Hide resolved
src/OrchardCore/OrchardCore.ContentManagement.GraphQL/Queries/ContentItemsFieldType.cs
Outdated
Show resolved
Hide resolved
var activator = typeActivator.GetTypeActivator(part.PartDefinition.Name); | ||
|
||
var queryGraphType = typeof(ObjectGraphType<>).MakeGenericType(activator.Type); | ||
var partType = PartTypes.GetOrAdd(part.PartDefinition.Name, key => typeActivator.GetTypeActivator(key).Type); |
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.
Now you are caching this one. So if the other one is not cache I assume it's because it doesn't have to?
src/OrchardCore/OrchardCore.ContentManagement.GraphQL/Queries/Types/TypedContentTypeBuilder.cs
Outdated
Show resolved
Hide resolved
test/OrchardCore.Tests/Apis/GraphQL/ContentItemsFieldTypeTests.cs
Outdated
Show resolved
Hide resolved
I will keep updating this PR with @hyzx86 |
#15215 has been merged and needs to be updated. |
This can be merged when the recent "current user" support has been handled |
This pull request has merge conflicts. Please resolve those before requesting a review. |
This pull request has merge conflicts. Please resolve those before requesting a review. |
@hyzx86 can you please fix the conflicts? |
I fixed the conflicts, hope this is correct. |
Seems there's fail in the build, @hyzx86 could you please re-check |
I fixed the build |
Co-authored-by: Hisham Bin Ateya <[email protected]> Co-authored-by: hyzx86 <[email protected]> Co-authored-by: Sébastien Ros <[email protected]> Co-authored-by: Mike Alhayek <[email protected]>
Make a note of any problems you encounter during the upgrade
GraphQL 7.7.2
Graphql-dotnet 7.x
recommendedFixes #15104