Skip to content
This repository has been archived by the owner on Dec 22, 2023. It is now read-only.

New plugin rofl #158

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

New plugin rofl #158

wants to merge 3 commits into from

Conversation

jurkov
Copy link
Contributor

@jurkov jurkov commented Nov 8, 2016

I just created the roflcopter command requested (#129) on the hacktoberfest event.

@izzy
Copy link
Member

izzy commented Nov 9, 2016

Those are quite a few lines that spits out. That could be problematic when it's used two or three times in a row and the bot either rate-limits itself or gets kicked for flooding.. :/

@jurkov
Copy link
Contributor Author

jurkov commented Nov 9, 2016

I think this is something really important for every plugin. Can we out source something like that to the bytebot irc engine?

@izzy
Copy link
Member

izzy commented Nov 10, 2016

IIRC irc3 already does rate-limit the privmsg to prevent flood-kicks, but with that many lines, it's probable that people will write inbetween the lines of the copter...

@mape2k mape2k changed the base branch from refactor-irc3 to master December 19, 2017 21:39
@mape2k
Copy link
Contributor

mape2k commented Dec 19, 2017

Plugin is now about an year old... IMHO breaking the output by users is not an problem as the output is just for fun.

@mkzero: Seems to be OK, we should merge (and have some fun).

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants