[
https://issues.apache.org/jira/browse/EAGLE-81?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15121478#comment-15121478
]
ASF GitHub Bot commented on EAGLE-81:
-------------------------------------
Github user yonzhang commented on the pull request:
https://github.com/apache/incubator-eagle/pull/77#issuecomment-176191482
Thanks for redoing pull request. I don't understand why we need bind
notificationType with AlertDefinitionAPIEntity. Doesn't
AlertDefinitionAPIEntity has a field notificationDef where notifications are
defined. Also, one alert definition should possibly have multiple notification
types associated.Do I miss anything? (I did not mention notificationDef to you
before, sorry about that)
> Notification Plugin Framework
> -----------------------------
>
> Key: EAGLE-81
> URL: https://issues.apache.org/jira/browse/EAGLE-81
> Project: Eagle
> Issue Type: New Feature
> Reporter: Senthilkumar
> Assignee: Senthilkumar
> Original Estimate: 336h
> Remaining Estimate: 336h
>
> Today notification is only allowed for email .. We need more notification
> methods to send alert to external system .
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)