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.

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

Attach user info to content. #6204

Closed
B-Esmaili opened this issue May 17, 2020 · 8 comments · Fixed by #7570
Closed

Attach user info to content. #6204

B-Esmaili opened this issue May 17, 2020 · 8 comments · Fixed by #7570

Comments

@B-Esmaili
Copy link

Is there any way to attach user info to content type? something like user field.allowing us to attach multiple user to instance of a content item.

@Piedone
Copy link
Member

Piedone commented May 17, 2020

To my knowledge, you need to implement this yourself, e.g. from a content part where you store the user's ID and then load it when necessary.

@B-Esmaili
Copy link
Author

The hard part is implementing a UserPicker . i have no idea how to implement that.

@Piedone
Copy link
Member

Piedone commented May 17, 2020

I suggest you look around in the AdminController class of OrchardCore.Users. You'll see how users are retrieved so you can then do the same in your own content part.

@deanmarcussen
Copy link
Member

We discussed this at the meeting last week https://www.youtube.com/watch?v=hexujWzGV14
and seemed to think it was something we should implement.

The code to look at for similarities on picker design would be the ContentPickerField

Probably the thing blocking implementing this in Orchard Core at the moment (for me) is the use of a int DocumentId property on the User object, as this will not carry over between different systems.

Can anyone in the team comment as to why we use the document id for the User rather than a Guid ?

(semi related #5409)

@kevinchalet I wonder if you had any involvement with the original choice there, or opinions on it?

@B-Esmaili
Copy link
Author

Glad to hear that. frankly i was disappointed about lack of this feature and i think it is something to be implement as builtin feature. actually the need for this feature originated from assigning user info to content items in workflow tasks and i think user specific tasks are lacking in workflow for that matter.

@jtkech
Copy link
Member

jtkech commented May 21, 2020

Can anyone in the team comment as to why we use the document id for the User rather than a Guid ?

@deanmarcussen sorry i don't know why the user id is the doc id so tied the the store. Looks good to have an uid added to the UserIndex table and then update the UserStore FindByIdAsync()

@Skrypt
Copy link
Contributor

Skrypt commented Jun 3, 2020

This issue is getting old. I should probably work on it since I'm concerned by this issue. This should be a requirement before shipping RC2 because afterward migrating all the user data and people module extensions might become more complex.

@B-Esmaili
Copy link
Author

RC2 is released but still no milestone specified.Have you done something regarding this?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants