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

MSC2313: Moderation policies as rooms (ban lists) #2313

Merged
merged 4 commits into from Nov 25, 2019
Merged

MSC2313: Moderation policies as rooms (ban lists) #2313

merged 4 commits into from Nov 25, 2019

Conversation

ara4n
Copy link
Member

@ara4n ara4n commented Oct 5, 2019

@ara4n ara4n added the proposal label Oct 5, 2019
Copy link
Member

@KitsuneRal KitsuneRal left a comment

Formatting and wording fixes.

proposals/2313-ban-lists.md Outdated Show resolved Hide resolved
proposals/2313-ban-lists.md Outdated Show resolved Hide resolved
proposals/2313-ban-lists.md Outdated Show resolved Hide resolved
proposals/2313-ban-lists.md Outdated Show resolved Hide resolved
proposals/2313-ban-lists.md Outdated Show resolved Hide resolved
proposals/2313-ban-lists.md Outdated Show resolved Hide resolved
proposals/2313-ban-lists.md Outdated Show resolved Hide resolved
@ara4n ara4n mentioned this pull request Oct 8, 2019
uhoreg
uhoreg approved these changes Oct 23, 2019
Copy link
Member

@uhoreg uhoreg left a comment

lgtm, modulo Kitsune's suggestions

@uhoreg
Copy link
Member

@uhoreg uhoreg commented Oct 24, 2019

@mscbot fcp merge

@mscbot
Copy link
Collaborator

@mscbot mscbot commented Oct 24, 2019

Team member @uhoreg has proposed to merge this. The next step is review by the rest of the tagged people:

No concerns currently listed.

Once at least 75% of reviewers approve (and none object), this will enter its final comment period. If you spot a major issue that hasn't been raised at any point in this process, please speak up!

See this document for info about what commands tagged team members can give me.

@mscbot mscbot added proposed-final-comment-period disposition-merge labels Oct 24, 2019
Co-Authored-By: Kitsune Ral <Kitsune-Ral@users.sf.net>
proposals/2313-ban-lists.md Outdated Show resolved Hide resolved
@turt2live turt2live changed the title MSC2313: Ban lists MSC2313: Moderation policies as rooms (ban lists) Nov 18, 2019
@mscbot mscbot added final-comment-period and removed proposed-final-comment-period labels Nov 18, 2019
@mscbot
Copy link
Collaborator

@mscbot mscbot commented Nov 18, 2019

🔔 This is now entering its final comment period, as per the review above. 🔔

@mscbot mscbot added finished-final-comment-period and removed final-comment-period labels Nov 25, 2019
@mscbot
Copy link
Collaborator

@mscbot mscbot commented Nov 25, 2019

The final comment period, with a disposition to merge, as per the review above, is now complete.

@turt2live turt2live merged commit 2a82a5c into master Nov 25, 2019
8 checks passed
@turt2live turt2live added spec-pr-missing and removed finished-final-comment-period labels Jan 8, 2020
@turt2live turt2live self-assigned this Feb 13, 2020
turt2live added a commit that referenced this issue Feb 13, 2020
@turt2live turt2live added spec-pr-in-review and removed spec-pr-missing labels Feb 13, 2020
@turt2live
Copy link
Member

@turt2live turt2live commented Feb 13, 2020

Spec PR: #2434

@turt2live turt2live added merged and removed spec-pr-in-review labels Feb 18, 2020
@turt2live
Copy link
Member

@turt2live turt2live commented Feb 18, 2020

Merged 🎉

@turt2live turt2live added the kind:core label Apr 20, 2020
* Is a room...
* Applied to a user: The user should leave the room and not join it
([MSC2270](https://github.com/matrix-org/matrix-doc/pull/2270)-style ignore).
* Applied to a room: No-op because a room cannot ban itself.

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Make this equivalent to tombstone instead.

a simple implementation is as follows:

* Is a user...
* Applied to a user: The user should be added to the subscriber's ignore list.

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Block list is more preferrable.

@matrix-org matrix-org locked as resolved and limited conversation to collaborators Nov 14, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
disposition-merge kind:core merged proposal
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

7 participants