search

TCCPP / wheatley

Wheatley discord bot for the Together C & C++ discord server
Other
13 stars 20 forks source link

Respond to users attempting to ping everyone #45

Closed Phytolizer closed 1 year ago

Phytolizer commented 1 year ago

Fix #22.

I would like some help landing on a better wording for the response. Also, maybe the event should be logged somewhere to improve mod awareness.

Phytolizer commented 1 year ago

I also feel the need to mention that it's my first time working with the components, so I might have screwed something up, but it seems pretty simple so I doubt it.

Phytolizer commented 1 year ago

How does the message look to you now? Maybe it would be better to stick to the one sentence.

Phytolizer commented 1 year ago

If I shorten it, it would probably make more sense as a regular message rather than an embed, too.

jeremy-rifkin commented 1 year ago

Interesting, in theory both this.wheatley.TCCPP.memberCount and this.wheatley.TCCPP.members.cache.size should work. The message looks good to me. Could go either way on the second sentence and embed question, dealer's choice :) I may lean slightly towards the one sentence. message.reply can be used for a text reply if you want, instead of adding an explicit mention.

Phytolizer commented 1 year ago

I'll go for the reply and single sentence.