-
Notifications
You must be signed in to change notification settings - Fork 380
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Proposal to improve /createRoom #3809
Comments
@Half-Shot and @jcgruenhage please be aware that the scope of this proposal has changed. Pinging because you've thumbed-up it so far. |
@turt2live I'd still say it's fine. |
I've got no issues with it either |
Actually, no, I don't think we need to require a state key here, defaulting to an empty state key here feels better. While the presence of a state key makes a state event, this API is supposed to build state events, so it is implied to exist. Given that https://matrix.org/docs/spec/client_server/r0.3.0.html#put-matrix-client-r0-rooms-roomid-state-eventtype-statekey says |
(note: we're arguing about this actively in #matrix-spec, proposal may be updated further) |
Documentation: https://docs.google.com/document/d/1HM5ANCyrqXVGVLERzZkYI_3x2_KpYYmtWZXnFdIV66k/edit?usp=sharing
PRs: #1326
Related: https://github.com/matrix-org/matrix-doc/issues/1331
The text was updated successfully, but these errors were encountered: