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

Add internal API QueryMembershipForUser #544

Merged
merged 2 commits into from
Jul 11, 2018

Conversation

APwhitehat
Copy link
Contributor

Lightweight API to check membership for an user in a room.

Signed-off-by: Anant Prakash <anantprakashjsr@gmail.com>
@APwhitehat APwhitehat force-pushed the membership_internalapi branch from 8ae57dd to c90e528 Compare July 10, 2018 20:25
// ID of the room to fetch membership from
RoomID string `json:"room_id"`
// ID of the user sending the request
Sender string `json:"sender"`
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Should this be changed from sender to UserID ?
since we want the membership for this user and not the sender.
@erikjohnston

Copy link
Member

Choose a reason for hiding this comment

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

Yeah, I think so

Copy link
Member

@erikjohnston erikjohnston left a comment

Choose a reason for hiding this comment

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

Other than renaming sender, looks good!

// ID of the room to fetch membership from
RoomID string `json:"room_id"`
// ID of the user sending the request
Sender string `json:"sender"`
Copy link
Member

Choose a reason for hiding this comment

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

Yeah, I think so

@APwhitehat APwhitehat merged commit c8feee7 into matrix-org:master Jul 11, 2018
@APwhitehat
Copy link
Contributor Author

merged on prior approval :P #544 (review)

@APwhitehat APwhitehat deleted the membership_internalapi branch July 11, 2018 10:11
anoadragon453 pushed a commit that referenced this pull request Jul 16, 2018
* Add internal API QueryMembershipForUser

Signed-off-by: Anant Prakash <anantprakashjsr@gmail.com>

* Change sender to user_id
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants