Skip to content

Add maySendEvent to match maySendStateEvent - #118

Merged
richvdh merged 2 commits into
developfrom
dbkr/maysendevent
Mar 24, 2016
Merged

Add maySendEvent to match maySendStateEvent#118
richvdh merged 2 commits into
developfrom
dbkr/maysendevent

Conversation

@dbkr

@dbkr dbkr commented Mar 23, 2016

Copy link
Copy Markdown
Member

Make them use the same function internally. Also add convenience maySendMessage. Also tests.

…unction internally. Also add convenience maySendMessage. Also tests.
Comment thread lib/models/room-state.js
* Returns true if the given user ID has permission to send a normal
* event of type `eventType` into this room.
* @param {string} type The type of event to test
* @param {string} userId The user ID of the user to test permission for

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

this doesn't exist

@richvdh

richvdh commented Mar 23, 2016

Copy link
Copy Markdown
Member

lgtm otherwise

@richvdh richvdh assigned dbkr and unassigned richvdh Mar 23, 2016
@richvdh
richvdh merged commit 69c4d7b into develop Mar 24, 2016
@dbkr
dbkr deleted the dbkr/maysendevent branch December 6, 2017 15:29
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.

2 participants