----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/27254/ -----------------------------------------------------------
Review request for Ambari, Aleksandr Kovalenko and Srimanth Gunturi. Repository: ambari Description ------- In this jira, need to make sure all new alerts pages are there with blank content (or some test content), also the URLs are correct. (For how new pages look like, take a look at the mock ups in https://hortonworks.jira.com/browse/RMP-1745.) Here are the new alerts pages we need to add : * Alerts summary page: a new tab 'Alerts' on the top-nav bar will trigger this page. So we will have 'Dashboard', 'Services', 'Hosts', 'Alerts', 'Admin', 'Views' ... on the top nav bar after adding 'Alerts'. The URL should be: http://<hostname>:8080/#/main/alerts * Alert definition page. The URL should be: http://<hostname>:8080/#/main/alerts/<definition_name> * Host details > Alerts tab. The URL should be: http://<hostname>:8080/#/main/hosts/<hostname>/summary Diffs ----- ambari-web/app/config.js f7171bf ambari-web/app/messages.js 99a429c ambari-web/app/routes/main.js bfdfc8c ambari-web/app/templates/main/alerts.hbs PRE-CREATION ambari-web/app/templates/main/host/alerts.hbs PRE-CREATION ambari-web/app/views.js 1970248 ambari-web/app/views/main/alerts.js PRE-CREATION ambari-web/app/views/main/host/alerts.js PRE-CREATION ambari-web/app/views/main/host/menu.js 0cd19b3 ambari-web/app/views/main/menu.js 6c68f53 Diff: https://reviews.apache.org/r/27254/diff/ Testing ------- Thanks, Xi Wang
