Skip to content
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.

Sign up for GitHub

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

Jump to bottom

The CustomUserSettings in the graphql me field is registered incorrectly #15540

Closed
hyzx86 opened this issue Mar 18, 2024 · 1 comment · Fixed by #15691
Closed

The CustomUserSettings in the graphql me field is registered incorrectly #15540

hyzx86 opened this issue Mar 18, 2024 · 1 comment · Fixed by #15691
Labels
bug 🐛
Milestone
2.0

Comments

@hyzx86
Copy link
Contributor

hyzx86 commented Mar 18, 2024

The type registration error occurred because the AddField method in the UserType type had a method error when registering the type

The fields in the red box below are not of the type for CustomUserSettings, they are actually CustomSettings.

image

this PR #15389, will fix the problem
@hyzx86
Copy link
Contributor Author

hyzx86 commented Mar 18, 2024

cc @gvkries

@hyzx86 hyzx86 mentioned this issue Mar 18, 2024
Merged
@agriffard agriffard changed the title The CustmUserSettings in the graphql me field is registered incorrectly The CustomUserSettings in the graphql me field is registered incorrectly Mar 21, 2024
@sebastienros sebastienros added this to the 1.x milestone Mar 21, 2024
This was referenced Apr 1, 2024
Closed
Open
gvkries added a commit to gvkries/OrchardCore that referenced this issue Apr 8, 2024
@gvkries
Makes the GraphQL user type reusable.
e18892e 
Fixes OrchardCMS#15540
@gvkries gvkries mentioned this issue Apr 8, 2024
Merged
@MikeAlhayek MikeAlhayek modified the milestones: 2.x, 2.0 Sep 7, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug 🐛
Projects
None yet
Milestone
2.0
Development

Successfully merging a pull request may close this issue.

Makes the GraphQL user type reusable. gvkries/OrchardCore
3 participants
@sebastienros @hyzx86 @MikeAlhayek