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

Implemented GraphQL query for retrieving current user data and its custom settings #15215

Merged
merged 2 commits into from
Feb 2, 2024
Merged

Implemented GraphQL query for retrieving current user data and its custom settings #15215

merged 2 commits into from
Feb 2, 2024

Conversation

gvkries
Copy link
Contributor

@gvkries gvkries commented Jan 31, 2024 •
edited
Loading

Fixes #15214

@hyzx86
Copy link
Contributor

hyzx86 commented Jan 31, 2024 •
edited
Loading

Good PR !👍

Is there a plan to implement UserPickerField?
You can refer to ContentPickerField

@gvkries
Copy link
Contributor Author

gvkries commented Jan 31, 2024

Is there a plan to implement UserPickerField? You can refer to ContentPickerField

I had no need for that at the moment, but it would be a good addition.

@hishamco
Copy link
Member

@hyzx86 can you review this PR?

@hyzx86
Copy link
Contributor

hyzx86 commented Jan 31, 2024

@hyzx86 can you review this PR?

I have seen, I also have plans to do this feature, but it seems that he is more perfect implementation, I have not considered the hidden type

@github-actions github-actions bot mentioned this pull request Feb 1, 2024
Closed
@sebastienros
Copy link
Member

Should this wait for the GraphQL package update to be merged? OR @hyzx86 do you want this one to be merged first and you to update your PR with that.

@hyzx86
Copy link
Contributor

hyzx86 commented Feb 2, 2024 •
edited
Loading

Should this wait for the GraphQL package update to be merged? OR @hyzx86 do you want this one to be merged first and you to update your PR with that.

That's not a big change. It can be merged into the main branch first.
Later I can add GraphQL support for UserPickerField

@sebastienros sebastienros merged commit ac9302d into OrchardCMS:main Feb 2, 2024
4 checks passed
@sebastienros sebastienros mentioned this pull request Feb 2, 2024
Merged
3 tasks
@gvkries gvkries deleted the gvkries/user-graphql-15214 branch February 3, 2024 13:24
urbanit pushed a commit to urbanit/OrchardCore that referenced this pull request Mar 18, 2024
@gvkries @urbanit
Implemented GraphQL query for retrieving current user data and its cu…
49c1dad 
…stom settings. (OrchardCMS#15215)
@hyzx86 hyzx86 mentioned this pull request May 14, 2024
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@hyzx86 hyzx86 hyzx86 approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

GraphQL - Query current user and custom user settings
4 participants
@gvkries @hyzx86 @hishamco @sebastienros