/synapse

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Already on GitHub? Sign in to your account

client/v1/room: include event_id in response to state event PUT #581

Merged
merged 1 commit into from Feb 17, 2016

Conversation

Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
3 participants
@Rugvip @matrixbot @erikjohnston
@Rugvip
Contributor

Rugvip commented Feb 17, 2016

Spec says that this should be done, so let's do it :>

https://matrix.org/docs/spec/r0.0.1/client_server.html#put-matrix-client-r0-rooms-roomid-state-eventtype
@Rugvip
client/v1/room: include event_id in response to state event PUT, in a… 
…ccordance with the spec

Signed-off-by: Patrik Oldsberg <patrik.oldsberg@ericsson.com>
71d5d2c
@matrixbot

matrixbot commented Feb 17, 2016

Can one of the admins verify this patch?
@erikjohnston
Owner

erikjohnston commented Feb 17, 2016

@matrixbot ok to test
@erikjohnston
Owner

erikjohnston commented Feb 17, 2016

LGTM, thanks :)

erikjohnston added a commit that referenced this pull request Feb 17, 2016

@erikjohnston
Merge pull request #581 from Rugvip/event_id 
client/v1/room: include event_id in response to state event PUT
97d1b3a

@erikjohnston erikjohnston merged commit 97d1b3a into matrix-org:develop Feb 17, 2016
1 check passed

1 check passed

default Merged build finished.
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment