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

Add new motion restriction modes #816

Closed
1 of 2 tasks
jsangmeister opened this issue Jan 18, 2024 · 2 comments · Fixed by #820
Closed
1 of 2 tasks

Add new motion restriction modes #816

jsangmeister opened this issue Jan 18, 2024 · 2 comments · Fixed by #820
Assignees
Milestone

Comments

@jsangmeister
Copy link
Contributor

jsangmeister commented Jan 18, 2024

see OpenSlides/openslides-backend#2119:

  • The new motion restriction mode B should only be visible to users with motion.can_manage

see OpenSlides/openslides-backend#2120:

  • TODO
@jsangmeister jsangmeister added this to the 4.2 milestone Jan 18, 2024
@jsangmeister jsangmeister changed the title Add new motion restriction mode Add new motion restriction modes Jan 18, 2024
@jsangmeister
Copy link
Contributor Author

For OpenSlides/openslides-backend#2120, we still need to decide on how to exactly model it. A possible way could be to implement this in the restricter the following way: To determine if the user can see the field motion/recommendation_id, the restricter follow the relation and checks the field motion_state/internal_recommendation. If that is true, the user needs motion.can_manage to see the recommendation. Otherwise, restriction mode C is used. @ostcar will evaluate how feasible this solution is or if there is a better way to model it.

@jsangmeister
Copy link
Contributor Author

extracted second bullet point to #840. Closing this.

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

Successfully merging a pull request may close this issue.

2 participants