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


Ship it!




Ship It!

- Jaimin Jetly


On July 22, 2016, 8:28 p.m., Manasi Maheshwari wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/50314/
> -----------------------------------------------------------
> 
> (Updated July 22, 2016, 8:28 p.m.)
> 
> 
> Review request for Ambari, Alejandro Fernandez, Jaimin Jetly, and Yusaku Sako.
> 
> 
> Bugs: AMBARI-17824
>     https://issues.apache.org/jira/browse/AMBARI-17824
> 
> 
> Repository: ambari
> 
> 
> Description
> -------
> 
> Following needs to be done as part of this work
> view instances needs to be shown under relevant service as tabs
> content of the view resources needs to be shown in iframe
> Quick links location needs to be adjusted so it does not collapse with view 
> tabs
> Left service menu on the service page needs to be removed to give more space 
> to view content
> Increase the span for the pages under service page for larger width
> 
> 
> Diffs
> -----
> 
>   ambari-web/app/app.js 79493a1 
>   ambari-web/app/messages.js 24c9cfa 
>   ambari-web/app/models/stack_service.js 3e2d5b4 
>   ambari-web/app/routes/main.js ebf9cb3 
>   ambari-web/app/styles/application.less 200c7be 
>   ambari-web/app/templates/main/service/item.hbs 43fb3dd 
>   ambari-web/app/views.js 904edb7 
>   ambari-web/app/views/common/widget/heatmap_widget_view.js 6c74420 
>   ambari-web/app/views/main/dashboard.js 9694774 
>   ambari-web/app/views/main/service/item.js 02ac426 
>   ambari-web/app/views/main/views/details.js df1b19b 
> 
> Diff: https://reviews.apache.org/r/50314/diff/
> 
> 
> Testing
> -------
> 
> I verified that the patch accomplishes the tasks written in the description.
> Verified that unit test passes with the patch
> 29343 tests complete (26 seconds)
> 154 tests pending
> 
> 
> Thanks,
> 
> Manasi Maheshwari
> 
>

Reply via email to