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

Room details: check power level before displaying actions in the timeline #3971

Merged
merged 3 commits into from
Sep 7, 2021

Conversation

Florian14
Copy link
Contributor

This PR fixes #3959

User power-level is now checked before showing action as set a topic or invite people to a room.
Previously, an error dialog was shown after the forbidden action, making a bad user experience.

@Florian14 Florian14 force-pushed the feature/fre/room_details_power_level branch from 0bbba30 to 991609a Compare September 6, 2021 14:40
Copy link
Member

@bmarty bmarty left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, thanks, just some remarks about naming

@Florian14 Florian14 force-pushed the feature/fre/room_details_power_level branch from 991609a to 32303af Compare September 7, 2021 09:43
Copy link
Member

@bmarty bmarty left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the update!

@bmarty bmarty merged commit c6a99f1 into develop Sep 7, 2021
@bmarty bmarty deleted the feature/fre/room_details_power_level branch September 7, 2021 10:09
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 this pull request may close these issues.

[Room details] Hide options when the user is not allowed to use them
3 participants