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

Community admins can delete user messages #8892

Merged
merged 2 commits into from
Jan 4, 2023

Conversation

jrainville
Copy link
Member

Fixes #8869 and #6891

Feature and test for the community admin being able to delete anyone's messages

@jrainville jrainville requested review from a team, iurimatias, borismelnik, glitchminer and noeliaSD and removed request for a team and iurimatias December 26, 2022 16:33
@status-im-auto
Copy link
Member

status-im-auto commented Dec 26, 2022

Jenkins Builds

Commit #️⃣ Finished (UTC) Duration Platform Result
✔️ cd98460 #1 2022-12-26 16:37:32 ~4 min imports 📄log
✔️ cd98460 #1 2022-12-26 16:37:50 ~4 min linux-cpp 📄log
✔️ cd98460 #1 2022-12-26 16:42:11 ~8 min macos 🍎dmg
✔️ cd98460 #1 2022-12-26 16:46:02 ~12 min linux 📦tgz
✔️ cd98460 #1 2022-12-26 16:59:13 ~25 min windows 💿exe
✖️ cd98460 #1 2022-12-26 17:03:08 ~30 min linux-e2e 📄log
✔️ cd98460 #2 2022-12-26 22:20:53 ~25 min linux-e2e 📄log

@jrainville jrainville requested review from a team, iurimatias, endulab and 0x-r4bbit and removed request for borismelnik, noeliaSD, a team and iurimatias January 3, 2023 14:56
Copy link
Member

@0x-r4bbit 0x-r4bbit left a comment

Choose a reason for hiding this comment

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

Nice one.

Copy link
Contributor

@mprakhov mprakhov left a comment

Choose a reason for hiding this comment

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

LGTM 👍

@jrainville jrainville merged commit 235c0d3 into master Jan 4, 2023
@jrainville jrainville deleted the feat/delete-messages-as-admin branch January 4, 2023 16:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Let admins delete messages from other users
4 participants