Skip to content

Add an option to add a delay between bot messages#106

Merged
ORelio merged 1 commit intoMCCTeam:Indevfrom
Pokechu22:add-bot-ratelimit
Jan 30, 2016
Merged

Add an option to add a delay between bot messages#106
ORelio merged 1 commit intoMCCTeam:Indevfrom
Pokechu22:add-bot-ratelimit

Conversation

@Pokechu22
Copy link
Contributor

Because bots can send several messages quickly, this adds an option to slow down the rate at which messages are produced (to avoid issues with antispam plugins). This should help solve part of the troubles @mobdon was having in #105.

Right now the default time is 2 seconds per message. However, messages are sent imediately if the bot doesn't need to delay (so if it's a bot that only outputs one or two messages, those will still happen imediately). Also, note that it's limited per-bot right now.

I also added an optional parameter to the SendText method so that bots can avoid this behavior if they need to. In some cases, they'll want to send multiple messages.

Because bots can send several messages quickly, this adds an option to slow down the rate at which messages are produced (to avoid issues with antispam plugins).  This should help solve part of the troubles @mobdon was having in MCCTeam#105.

Right now the default time is 2 seconds per message.  However, messages are sent imediately if the bot doesn't need to delay (so if it's a bot that only outputs one or two messages, those will still happen imediately).  Also, note that it's limited per-bot right now.

I also added an optional parameter to the SendText method so that bots can avoid this behavior if they need to.  In some cases, they'll want to send multiple messages.
@Pokechu22 Pokechu22 changed the title Add an option to add a delay between bot message Add an option to add a delay between bot messages Jan 30, 2016
@ORelio
Copy link
Member

ORelio commented Jan 30, 2016

Pretty neat. Thanks :)

ORelio added a commit that referenced this pull request Jan 30, 2016
Add an option to add a delay between bot messages
@ORelio ORelio merged commit 73a1645 into MCCTeam:Indev Jan 30, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants