unexBot icon indicating copy to clipboard operation
unexBot copied to clipboard

Users with Whitelisted PMs don't get the message.

Open Psyda opened this issue 3 years ago • 2 comments

It spits out an error when trying to send a PM about them having whitelisted PMs and the user is oblivious after. I've seen it happen several times. It just deletes their post after the time without them knowing why.

I'd consider a solution to this being:

Forgo the PMs and instead have the bot post a comment instead. Once the user replies with the reasoning delete both the comment and response and post a pinned top-level comment.

Psyda avatar May 04 '21 18:05 Psyda

I don't think the PMs should be foregone entirely. Maybe add an exception instead that can use the alternate solution when users don't have their PMs public.

That nasty [deleted] chain would get ugly on every post, since you'd get two of those for every post. Probably best to only do it when absolutely necessary.

miike3459 avatar May 05 '21 12:05 miike3459

I'm not sure how urgent this issue is, since only very few people have their PMs disabled, but I guess that implementing an alternative communication strategy would be reasonable.

I'm planning on expanding the capabilities and the flexibility of the bot anyway, alongside a lot of refactoring. I'm planning to rename what is now the Conversation Interface to an Interaction Interface and allow different implementations of that. In this case that would be a "CommentInteraction" that would be fired when the messaging fails.

I'd have to investigate where the exception is thrown though, do you have a stacktrace that you could post here?

Artraxon avatar May 05 '21 12:05 Artraxon