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.

Sign up for GitHub

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

Jump to bottom

Fix membership state table and diagram #3730

Merged
merged 4 commits into from
Feb 22, 2022
Merged

Fix membership state table and diagram #3730

merged 4 commits into from
Feb 22, 2022

Commits on Feb 18, 2022

  1. Fix membership state table and diagram 

    There were 2 missing cases which are legal:
* `invite->knock` (a fairly silly thing to do, but legal under the auth rules)
* `external->leave (via /kick)` (another somewhat silly thing to do, but no different than `external->ban (via /ban)`)

The state table considered the first as illegal, which is untrue.
    @turt2live
    turt2live committed Feb 18, 2022
    Configuration menu
    Copy the full SHA
    7075a59 View commit details
    Browse the repository at this point in the history

  2. Changelog

    @turt2live
    turt2live committed Feb 18, 2022
    Configuration menu
    Copy the full SHA
    65f844d View commit details
    Browse the repository at this point in the history

  3. Make the graph prettier

    @turt2live
    turt2live committed Feb 18, 2022
    Configuration menu
    Copy the full SHA
    758a6d4 View commit details
    Browse the repository at this point in the history

Commits on Feb 22, 2022

  1. Update changelogs/client_server/newsfragments/3730.clarification 

    Co-authored-by: Richard van der Hoff <1389908+richvdh@users.noreply.github.com>
    @turt2live @richvdh
    turt2live and richvdh committed Feb 22, 2022
    Configuration menu
    Copy the full SHA
    1edbfbc View commit details
    Browse the repository at this point in the history