Skip Menu |

This queue is for tickets about the Darcs-Notify CPAN distribution.

Report information
The Basics
Id: 42215
Status: resolved
Priority: 0/
Queue: Darcs-Notify

People
Owner: david [...] cpan.org
Requestors: MARKSTOS [...] cpan.org
Cc:
AdminCc:

Bug Information
Severity: Wishlist
Broken in: (no value)
Fixed in: (no value)



Subject: wish: split out e-mail logic
Thanks for sharing this darcs tool. My wish is that "Darcs::Notify" and "Darcs::Notify::Email" be split out. In the parent class, you would do things that any notification method woudl want, like calculating the "diff" and "diffstat". These functions are currently tied to sending the e-mail. My wish is for it to be extremely easy to add new subclasses, like Darcs::Notify::IRC or Darcs::Notify::RSS. Thanks! Mark
I redid the API (I had been planning on it anyway) and it's now possible to extend Darcs::Notify to send different types of notifications. It maybe uses a little more magic than was strictly necessary, but that made it fun to write. ;-)