-
Notifications
You must be signed in to change notification settings - Fork 4
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
chat: add async rooms spec #217
Conversation
8ebed8d
to
3b576d3
Compare
Looks like there are some commits not related to the change above, it'll make this difficult to review. Is it possible to review, could we have a branch with the specific spec changes please? |
Yep - just need to rebase now the big one has landed. Will do that now! |
5eca441
to
b32a74a
Compare
b32a74a
to
9fffff0
Compare
9fffff0
to
1ca63a2
Compare
This change adds the specification for async room get in Chat.
1ca63a2
to
f3aa677
Compare
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.
Few small wording changes, but looks good.
This change adds the specification for async room get in Chat.
It also makes the room lifecycle changes in relation to CHADR-062.
CHA-669
CHA-637