issue 22: advertise/broadcast message

mozilla / testdaybot

Mozilla QA Test Day IRC Bot

9 stars 12 forks source link

issue 22: advertise/broadcast message #35

Closed galgeek closed 9 years ago

galgeek commented 9 years ago

hi @whimboo !

this pull request includes the issue 28 updates. the updates particular to issue 22 are those whose commit message begins issue 22. I'm not sure how best to test this. Is it ok to create a few test channels? But this code is also in fennecbot, hardly changed. And perhaps you'd prefer I resubmit after you merge issue 28 updates?

galgeek commented 9 years ago

hi @whimboo !

thanks for your feedback. please refer to #38 for my updates, and look above for some responses to your line notes. I'll leave this open for now to facilitate reviewing and close it later, or you're welcome to close it.

whimboo commented 9 years ago

As said we now cover this in PR #38.