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

Account deactivation: Add info on leaving rooms #9005

Closed
wants to merge 2 commits into from

Conversation

fadenb
Copy link

@fadenb fadenb commented Jan 1, 2021

Deactivating an account - in addition to the already listed things - also makes the user leave all rooms they are in.

Pull Request Checklist

  • Pull request is based on the develop branch
  • Pull request includes a changelog file. The entry should:
    • Be a short description of your change which makes sense to users. "Fixed a bug that prevented receiving messages from other servers." instead of "Moved X method from EventStore to EventWorkerStore.".
    • Use markdown where necessary, mostly for code blocks.
    • End with either a period (.) or an exclamation mark (!).
    • Start with a capital letter.
  • Pull request includes a sign off
  • Code style is correct (run the linters)

Signed-off-by: FadenB [email protected]

Deactivating an account - in addition to the already listed things - also makes the user leave all rooms they are in.
@dklimpel
Copy link
Contributor

dklimpel commented Jan 1, 2021

This information is also already part of #8932

@fadenb fadenb marked this pull request as ready for review January 1, 2021 13:25
@clokep
Copy link
Member

clokep commented Jan 4, 2021

This would need develop merged into to fix the build.

This information is also already part of #8932

I see no reason to not improve the docs before that's landed! 👍

@clokep clokep added the X-Awaiting-Changes A contributed PR which needs changes and re-review before it can be merged label Jan 4, 2021
@clokep
Copy link
Member

clokep commented Jan 27, 2021

It seems that #8932 was merged in the meantime so I'm going to close this. Thanks for your contribution!

@clokep clokep closed this Jan 27, 2021
@fadenb fadenb deleted the docs_user_admin_api branch January 28, 2021 10:07
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
X-Awaiting-Changes A contributed PR which needs changes and re-review before it can be merged
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants