-
-
Notifications
You must be signed in to change notification settings - Fork 832
Conversation
Add a specific dialog used for confirming member actions. Also remove onFinished from MemberInfo which did absolutely nothing.
depending on whether the user is banned already Mostly gives some feedback that the ban has actually taken effect.
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 apart from a rename
).then( | ||
let promise; | ||
if (this.props.member.membership == 'ban') { | ||
promise = this.props.matrixClient.unban( |
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.
onBan
should probably have a different name given that it might unban someone
@@ -481,22 +489,22 @@ module.exports = WithMatrixClient(React.createClass({ | |||
}, | |||
|
|||
_calculateCanPermissions: function(me, them, powerLevels) { |
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.
/me wishes this or something more generic for figuring out the client's abilities was pushed way further down the stack, potentially to the js-sdk
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.
Agreed this should definitely be in the js-sdk. Trying to put some limit on the depth of this rabbit hole traversal though.
ptal |
depending on whether the user is banned already
Mostly gives some feedback that the ban has actually taken effect.
Includes #694 so either merge that first & merge in, or just look at the last commit.