-
Notifications
You must be signed in to change notification settings - Fork 46
Fix non meeting members from being part of meeting #5692
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
Fix non meeting members from being part of meeting #5692
Conversation
|
amendments create cannot be tested right now |
Should be fixed now |
luisa-beerboom
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Two minor things, but generally I approve.
...list/components/chat-group-detail-message-form/chat-group-detail-message-form.component.html
Outdated
Show resolved
Hide resolved
...src/app/gateways/repositories/motions/motion-repository.service/motion-repository.service.ts
Outdated
Show resolved
Hide resolved
|
Also @bspekker , could you test if this fixes OpenSlides/openslides-backend#3133 ? |
bspekker
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM!
OpenSlides/openslides-backend#3133 is fixed as well.
resolves #5643
needs OpenSlides/openslides-backend#3206
Additionally:
This Issue could be fixed by this:
Locked Out Participant: error with deletion. users can't be orga/superadmins openslides-backend#3133