-
Notifications
You must be signed in to change notification settings - Fork 34
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
Update motion submission date label visiblity #4428
Update motion submission date label visiblity #4428
Conversation
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.
The motion needs a reload if the permissions from the participant change while the motion is already loaded
This seems to be a missing subscription, the Edit icon has the same problem, but that was already an Issue. If possible please also fix that part. If that's not possible please clarify, so an Issue for that can be created
Repro:
The submission date (if empty) and the icon should vanish without the need for a reload |
Changing the permission don't have a instant effect on another browser. This would be a large feature. |
Given that this behaviour is a bug and only appears/ becomes visible in this PR, this should be fixed in this PR As far as I can see permissions and what the user is allowed to see has to be loaded and updated in the same manner as e.g. the motion block in the motion detail page |
Added a change to the permission check of this component. |
...il/pages/motion-view/components/motion-manage-timestamp/motion-manage-timestamp.component.ts
Outdated
Show resolved
Hide resolved
@Elblinator I followed your suggestion and updated the perms in the timestamp component. |
Resolve #4393