Page MenuHomePhabricator

Figure out the correct way to trigger the Article Reminder event for all notification types
Closed, InvalidPublic

Event Timeline

By 'notification types', do you mean 'web' and 'email'? (The terminology here is poorly standardized, but you can call it a "notification method" or "delivery method". Notification type is something else).

I think this will Just Work.

By default, all notification types (e.g. mention, user talk, reverted, etc.) support both web and email. You trigger once with EchoEvent::create() (after defining the model), and Echo takes care of the delivery.

"Needs Review" means there is an associated patch.

I don't think there's a patch for this, but I don't think there needs to be. You can probably close this as Invalid, but if not please clarify the status.