backmeupplz/banofbot
Watch
1
0
Code Issues Pull requests Projects Releases Wiki Activity Actions

Fixed missactivation #8

Closed
TEXHIK wants to merge 1 commit from master into master
pull from: master
merge into: backmeupplz:master
Conversation 2 Commits 1 Files changed 1 +1 -1
TEXHIK commented 2018-08-15 08:39:03 +00:00 (Migrated from github.com)
Copy link

Bot should not be activated if it was not mentioned directly or if it wasn't a command.
For example, if we discuss something about banofbot, and use reply, we'll get activation.

Bot should not be activated if it was not mentioned directly or if it wasn't a command. For example, if we discuss something about banofbot, and use reply, we'll get activation.
backmeupplz commented 2018-08-15 20:10:22 +00:00 (Migrated from github.com)
Copy link

This is a feature, not an issue.

This is a feature, not an issue.
Mart-Bogdan commented 2018-08-17 07:18:41 +00:00 (Migrated from github.com)
Copy link

@backmeupplz if that's a feature, it should be ability to disable it through configuration

@backmeupplz if that's a feature, it should be ability to disable it through configuration

Pull request closed

Please reopen this pull request to perform a merge.
Sign in to join this conversation.
Reviewers
No reviewers
Labels
Clear labels   
bug

   
dependencies
Pull requests that update a dependency file

  
duplicate

   
enhancement

   
help wanted

   
invalid

   
javascript
Pull requests that update javascript code

  
question

   
wontfix
No labels
bug
dependencies
duplicate
enhancement
help wanted
invalid
javascript
question
wontfix
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set

Reference
backmeupplz/banofbot!8
No description provided.