Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Replace Pushover module with Notify #11

Closed
jacroe opened this issue Sep 8, 2012 · 1 comment
Closed

Replace Pushover module with Notify #11

jacroe opened this issue Sep 8, 2012 · 1 comment

Comments

@jacroe
Copy link
Owner

jacroe commented Sep 8, 2012

Replace the current Pushover module with a Notify module. This will be where we send out notifications by email. Pushover, or through the web interface (TBD how later). Something like:

function alice_notify(string $title, string $message, array $services)
#Services:
function alice_pushover(string $title, string $message)
function alice_notificationEmail(string $subject, string $body)
function alice_notificationWeb(string $subject, string $message)

Then using those listed in $services, make calls to each function.

(Hopefully what I'm trying to say will make sense later when I actually implement this.)

@jacroe
Copy link
Owner Author

jacroe commented Oct 12, 2012

No reason to have one function that simply calls other functions. It will be probably just as well that we call them individually. Won't implement.

@jacroe jacroe closed this as completed Oct 12, 2012
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant