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

Bot should PM players any unacknowledged warnings when trying to join #236

Open
ghost opened this issue Jul 26, 2016 · 3 comments
Open

Bot should PM players any unacknowledged warnings when trying to join #236

ghost opened this issue Jul 26, 2016 · 3 comments

Comments

@ghost
Copy link

ghost commented Jul 26, 2016

Suggested by @Michagogo on IRC.

@ghost ghost added the enhancement label Jul 26, 2016
@ghost ghost assigned skizzerz Jul 26, 2016
@skizzerz skizzerz removed their assignment Jul 26, 2016
@Vgr255
Copy link
Member

Vgr255 commented Jul 26, 2016

IMO, making the players having to do !warn list helps ensure that they really know their warning (and don't just blindly do !ack # where # is just whatever they see in PM. But maybe that's not necessary, I don't know. I'm +0 anyway.

@Michagogo
Copy link

Well, on one hand that may be true, but I feel like sending the full
warning text in the message won't really hurt the issue. In fact, it may
even improve it. I feel like the player may be more likely to actually read
the message if it's sent to him in order to understand what's going on.
With the current one, I have a feeling people will read the message, see
what's happening, and at that point (before seeing the warning itself) go
into clickthrough/"clear this"/"get out of the way" mode and be more likely
to list just to find the number.

On Tuesday, 26 July 2016, Emanuel Barry [email protected] wrote:

IMO, making the players having to do !warn list helps ensure that they
really know their warning (and don't just blindly do !ack # where # is
just whatever they see in PM. But maybe that's not necessary, I don't know.
I'm +0 anyway.


You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
#236 (comment), or mute
the thread
https://github.com/notifications/unsubscribe-auth/ACcNnstAfciqjm1jwAyxX2bOQFQJSU6oks5qZmaDgaJpZM4JVgdA
.

@Fudster
Copy link
Contributor

Fudster commented Jan 17, 2018

I agree with vgr. But if the repo maintainers want this I can work on it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants