add actionable_message_for_me? to MessageHelper

rmcafee / discord_ex

Discord Elixir Library

MIT License

47 stars 13 forks source link

add actionable_message_for_me? to MessageHelper #9

Closed Marqin closed 8 years ago

Marqin commented 8 years ago

This adds actionable_message_for_me? that compares current user ID with list of mentioned IDs (and also checks DM).

pros:

you don't need to provide your bot name here
it's more reliable, because names can be changed (and spoofed)

Marqin commented 8 years ago

Also it would be nice to make old actionable_message_for more reliable (eg. check name and discriminator (that 4 digits after nick) ), but that would break compability, so plan it for 2.0

Marqin commented 8 years ago

Ok, this is not efficient, because it checks for bot ClientID every time (and acording to people ClientID cannot change during connection). I'm gona redo it.