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

Allow moderators to see who deleted a message #2874

Merged
merged 7 commits into from Jun 19, 2021
Merged

Allow moderators to see who deleted a message #2874

merged 7 commits into from Jun 19, 2021

Conversation

Mm2PL
Copy link
Collaborator

@Mm2PL Mm2PL commented Jun 7, 2021 •
edited

Pull request checklist:

  • CHANGELOG.md was updated, if applicable

Description

Adds support for pubsub deletion messages. Closes #2867.

@Mm2PL
Copy link
Collaborator Author

Mm2PL commented Jun 8, 2021 •
edited

Minor bug. I am pretty sure this is not in my code. Probably relates to #2804.

2021-06-08_13-58

Edit: This might be my code? i noticed also connected messages get removed.
fixed.

@Mm2PL Mm2PL mentioned this pull request Jun 9, 2021
Open
@Mm2PL
Copy link
Collaborator Author

Mm2PL commented Jun 10, 2021

@pajlada PRogramming content Okayge? Also I can't request review directly

@zneix zneix requested a review from pajlada June 10, 2021 15:30
pajlada
pajlada approved these changes Jun 19, 2021
View reviewed changes
Copy link
Member

@pajlada pajlada left a comment

Choose a reason for hiding this comment

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

👍

@pajlada pajlada enabled auto-merge (squash) June 19, 2021 11:29
@pajlada pajlada merged commit 9640837 into Chatterino:master Jun 19, 2021
@pajlada pajlada deleted the feature/pubsub_single_message_deletion_for_moderators branch June 19, 2021 11:56
zneix added a commit to SevenTV/chatterino7 that referenced this pull request Jun 19, 2021
@zneix
Merge branch 'master' into chatterino7
6568208 
Now we're on commit d6b5921; Changes from upstream we pulled:

- Major: Added username autocompletion popup menu when typing usernames with an @ prefix. (Chatterino#1979, Chatterino#2866)
- Minor: The /live split now shows channels going offline. (Chatterino#2880)
- Minor: Now shows deletions of messages like timeouts (Chatterino#1155, Chatterino#2841, Chatterino#2867, Chatterino#2874)
- Bugfix: Moderation buttons now show the correct time unit when using units other than seconds. (Chatterino#1719, Chatterino#2864)
- Bugfix: Fixed bit and new subscriber emotes not (re)loading in some rare cases. (Chatterino#2856, Chatterino#2857)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@zneix zneix zneix left review comments

@pajlada pajlada pajlada approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Show which mod deleted message
3 participants
@Mm2PL @pajlada @zneix