You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@ambari.apache.org by Xi Wang <xw...@hortonworks.com> on 2014/11/04 22:46:02 UTC

Re: Review Request 27433: Alerts UI: Create model and mapper for Alert-Notifications

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/27433/
-----------------------------------------------------------

(Updated Nov. 4, 2014, 1:46 p.m.)


Review request for Ambari, Aleksandr Kovalenko and Srimanth Gunturi.


Bugs: AMBARI-8055
    https://issues.apache.org/jira/browse/AMBARI-8055


Repository: ambari


Description
-------

Alert notifications will be used by Alert definition and host level alert pages. For them we need to know more details about the notifications that were used. 

We need a model and mapper created for Alert Notifications. We need to refer to the API to see what exact fields have to mapped.


{{Alert-Target}} class is:
* id (number),
* name: (string),
* type: (string),  // notification type 'EMAIL' ...
* description: (string),
* properties: (string)


Diffs (updated)
-----

  ambari-web/app/assets/data/alerts/alertNotifications.json PRE-CREATION 
  ambari-web/app/mappers/alert_notification_mapper.js PRE-CREATION 
  ambari-web/app/models.js b01fbc2 
  ambari-web/app/models/alert_instance.js b13b952 
  ambari-web/app/models/alert_notification.js PRE-CREATION 
  ambari-web/app/utils/ajax/ajax.js cb3bc28 

Diff: https://reviews.apache.org/r/27433/diff/


Testing
-------


Thanks,

Xi Wang