Skip to content
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

Spec soft-logout per MSC1466 #2546

Merged
merged 7 commits into from May 26, 2020
Merged

Spec soft-logout per MSC1466 #2546

merged 7 commits into from May 26, 2020

Conversation

turt2live
Copy link
Member

@turt2live turt2live commented May 15, 2020

MSC: #1466

@turt2live turt2live requested a review from May 15, 2020
@turt2live turt2live mentioned this pull request May 15, 2020
uhoreg
uhoreg previously requested changes May 16, 2020
specification/client_server_api.rst Outdated Show resolved Hide resolved
specification/client_server_api.rst Outdated Show resolved Hide resolved
@turt2live turt2live requested a review from uhoreg May 19, 2020
Copy link
Member

@uhoreg uhoreg left a comment

Suggested wording change, but lgtm otherwise. May also be worth getting a review from someone from the backend team.

specification/client_server_api.rst Outdated Show resolved Hide resolved
Co-authored-by: Hubert Chathi <hubert@uhoreg.ca>
@turt2live turt2live requested a review from May 19, 2020
@turt2live turt2live dismissed uhoreg’s stale review May 22, 2020

review status has changed

@turt2live turt2live added the next-release label May 25, 2020
Copy link
Member

@anoadragon453 anoadragon453 left a comment

lgtm otherwise

api/client-server/registration.yaml Outdated Show resolved Hide resolved
Co-authored-by: Andrew Morgan <1342360+anoadragon453@users.noreply.github.com>
@turt2live turt2live merged commit 04a26da into master May 26, 2020
0 of 6 checks passed
@turt2live turt2live deleted the travis/spec/1466-softlogout branch May 26, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
next-release
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants