Skip to content
This repository has been archived by the owner on Sep 11, 2024. It is now read-only.

Check permission to invite before showing invite buttons/disable them #2957

Merged
merged 2 commits into from
May 14, 2019

Conversation

t3chguy
Copy link
Member

@t3chguy t3chguy commented May 11, 2019

Fixes element-hq/element-web#9664

Signed-off-by: Michael Telatynski 7t3chguy@gmail.com

Signed-off-by: Michael Telatynski <7t3chguy@gmail.com>
@t3chguy t3chguy requested a review from a team May 12, 2019 16:15
@@ -87,6 +87,11 @@ limitations under the License.
}
}

.mx_MemberList_invite.mx_AccessibleButton_disabled {
background-color: grey;
Copy link
Member

Choose a reason for hiding this comment

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

There should be a variable for this, and if not, please make one.

Signed-off-by: Michael Telatynski <7t3chguy@gmail.com>
@t3chguy t3chguy requested a review from turt2live May 14, 2019 22:03
Copy link
Member

@turt2live turt2live left a comment

Choose a reason for hiding this comment

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

thanks!

edit: will wait for build before merging

@t3chguy
Copy link
Member Author

t3chguy commented May 14, 2019

built :)

@t3chguy t3chguy merged commit 9b40913 into develop May 14, 2019
@t3chguy t3chguy deleted the t3chguy/invite_perms branch May 25, 2020 18:08
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Invite button should be disabled for rooms not allowing invites
2 participants