Skip to content
This repository has been archived by the owner on Feb 23, 2021. It is now read-only.

broadcast should not send messages to all the users #35

Open
gabrielrussoc opened this issue Jan 21, 2018 · 1 comment
Open

broadcast should not send messages to all the users #35

gabrielrussoc opened this issue Jan 21, 2018 · 1 comment
Labels
bug urgent

Comments

@gabrielrussoc
Copy link
Member

right now we send the broadcast message to everyone, even to those who stopped (/stop) the bot
@gabrielrussoc
Copy link
Member Author

This breaks the logging. Since it triggers so many errors (because of users who blocked the bot) the log grows very quickly and somehow the Winston library doesn't seem to handle it very well.

pedroteosousa added a commit that referenced this issue Mar 19, 2018
@pedroteosousa
Only broadcasting to users that enabled notifications
5839b0d 
Disabling notifications for users that block or kick the bot

Fixes part of issue #35
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
bug urgent
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@gabrielrussoc