----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/29454/#review66711 -----------------------------------------------------------
Ship it! Ship It! - Andrii Tkach On Дек. 29, 2014, 1:44 п.п., Richard Zang wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/29454/ > ----------------------------------------------------------- > > (Updated Дек. 29, 2014, 1:44 п.п.) > > > Review request for Ambari, Andrii Tkach and Yusaku Sako. > > > Bugs: AMBARI-8937 > https://issues.apache.org/jira/browse/AMBARI-8937 > > > Repository: ambari > > > Description > ------- > > Create/Edit page layout adjustment. > Only show active version on create page. > Adjust repo version error input UI. > Provide link for cluster. > Adjust "Configure on..." to jump directly to specific repo version. > Adjust version filter on listing page. > Remove "Detail" section on edit page. > Adjust repo version shown on edit page breadcrum. > > > Diffs > ----- > > > ambari-admin/src/main/resources/ui/admin-web/app/scripts/controllers/stackVersions/StackVersionsCreateCtrl.js > 0d67f35 > > ambari-admin/src/main/resources/ui/admin-web/app/scripts/controllers/stackVersions/StackVersionsListCtrl.js > 0744a9a > > ambari-admin/src/main/resources/ui/admin-web/app/scripts/services/Cluster.js > a0e4ab6 > ambari-admin/src/main/resources/ui/admin-web/app/scripts/services/Stack.js > 4e93854 > ambari-admin/src/main/resources/ui/admin-web/app/styles/main.css a565534 > > ambari-admin/src/main/resources/ui/admin-web/app/views/stackVersions/create.html > c9f2668 > > ambari-admin/src/main/resources/ui/admin-web/app/views/stackVersions/edit.html > e4ae991 > > ambari-admin/src/main/resources/ui/admin-web/app/views/stackVersions/list.html > ef15f1d > > Diff: https://reviews.apache.org/r/29454/diff/ > > > Testing > ------- > > Manually tested on live cluster. Unit test passed. > > > Thanks, > > Richard Zang > >
