[ 
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)

Reply via email to