Skip to content

feat(plugin): HideMessage#2886

Open
UncreativeXenon wants to merge 20 commits into
Vendicated:mainfrom
UncreativeXenon:main
Open

feat(plugin): HideMessage#2886
UncreativeXenon wants to merge 20 commits into
Vendicated:mainfrom
UncreativeXenon:main

Conversation

@UncreativeXenon
Copy link
Copy Markdown

self-explanatory

image

image

@UncreativeXenon
Copy link
Copy Markdown
Author

Added a "Message hidden" text

image

@Feathers8
Copy link
Copy Markdown

I just want to mention that there is another pull request open for a hide message plugin.
#2860

@UncreativeXenon
Copy link
Copy Markdown
Author

Ok so.. after going back through the previous rejected PRs I've come across #2207 (comment)

  • dispatch MESSAGE_DELETE events for the hidden messages

I've decided to erase everything I did and implement this method instead.

@UncreativeXenon
Copy link
Copy Markdown
Author

I removed the saved ids thing as it seems useless to this, willing to hear suggestions though!

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.

2 participants