Code52 / jibbr

A Jabbr bot designed for collaborative projects
code52.org
67 stars 27 forks source link

Added Enable/Disable hook for sprockets #73

Closed brooklynDev closed 12 years ago

brooklynDev commented 12 years ago

The ISprocket interface now has an Enabled property. In the Bot, this property is set to true when starting up. Whenever the bot is sent a message, while it's looping through the list of ISprockets, it checks this flag. The bot now also has Enable/Disable methods to enable/disable specific sprockets. For now it just takes in a string (ISprocket.GetType().FullName) as there's no easy way to identify a Sprocket (maybe we should add a "SprocketName" or something, food for thought....). I also updated the AspNetBotHost web app with Enable/Disable buttons on the Index page alongside the Sprockets listed.

cyberzed commented 12 years ago

The things I'm adding in #69 could be used for it

brooklynDev commented 12 years ago

OK, I like the name idea. How do I go about merging your changes into mine? I'm still new to Git...

cyberzed commented 12 years ago

I had some issue with my commit and some lineend inconsistency, when I've added mine I'll send a pullrequest and then @shiftkey or one of the other will handle it, afterwards you can tune your stuff to use it.

shiftkey commented 12 years ago

I'm going to have to close this one - we've nearly got JibbR back to how it was, but still requires us to think about how it is hosted.

shiftkey commented 12 years ago

Ping @cyberzed