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

Allow meetings admins to change personal information for certain users #3841

Closed
Elblinator opened this issue Jul 11, 2024 · 3 comments · Fixed by #4205
Closed

Allow meetings admins to change personal information for certain users #3841

Elblinator opened this issue Jul 11, 2024 · 3 comments · Fixed by #4205
Assignees
Labels
Milestone

Comments

@Elblinator
Copy link
Member

see for context: OpenSlides/openslides-backend#2522

Current behaviour:
If a user (Ann) is a meeting admin (more precise when Ann is allowed to manage participants) in two meetings and another user (Ben) is only in these two meetings then Ann is currently not allowed to change the personal information from this user.
Currently Ann can still change the personal information but not save it. After the PR #3833 is mergen Ann is not even allowed to change the information in the edit mask from this user.

Wanted behaviour:
As long as Ann is allowed to manage participants in every meeting Ben is a participant in then Ann is allowed to change Ben's personal information

@Elblinator
Copy link
Member Author

The presenter was changed, please look into the backend PR/ descriptions for more details

@reiterl
Copy link
Member

reiterl commented Sep 30, 2024

What is the name of this new presenter? Then I can look at the presenter documentation.

@reiterl reiterl self-assigned this Sep 30, 2024
@reiterl
Copy link
Member

reiterl commented Oct 1, 2024

The new presenter is the get_user_editable presenter.
All personal information are of the same permission group in the user.update action (Group A).
This issue is only about the participant-detail view.

I think I will include the presenter in the UserService with one id and just a represent of the group A fields
and disable the Personal Information fields.

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

Successfully merging a pull request may close this issue.

2 participants