Re: Review Request 52535: Advanced storm-atlas-application.properties panel is not opened by default upon filtering

2016-10-05 Thread Zhe (Joe) Wang

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


Ship it!




Ship It!

- Zhe (Joe) Wang


On Oct. 4, 2016, 11:46 p.m., Vivek Ratnavel Subramanian wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/52535/
> ---
> 
> (Updated Oct. 4, 2016, 11:46 p.m.)
> 
> 
> Review request for Ambari, Jaimin Jetly, Zhe (Joe) Wang, Richard Zang, Xi 
> Wang, and Yusaku Sako.
> 
> 
> Bugs: AMBARI-18534
> https://issues.apache.org/jira/browse/AMBARI-18534
> 
> 
> Repository: ambari
> 
> 
> Description
> ---
> 
> - Navigate to Storm configs page
> - Type "hook" in the property filter
> - Its expected that Advanced storm-atlas-application.properties panel opens 
> up by default
> - Actual: its still closed. please see the screenshot attached
> 
> 
> Diffs
> -
> 
>   ambari-web/app/views/common/configs/service_configs_by_category_view.js 
> 3eebfea 
> 
> Diff: https://reviews.apache.org/r/52535/diff/
> 
> 
> Testing
> ---
> 
> Verified manually.
> Ambari-web unit test pass.
> 30374 tests complete (27 seconds)
> 151 tests pending
> 
> 
> Thanks,
> 
> Vivek Ratnavel Subramanian
> 
>



Review Request 52535: Advanced storm-atlas-application.properties panel is not opened by default upon filtering

2016-10-04 Thread Vivek Ratnavel Subramanian

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

Review request for Ambari, Jaimin Jetly, Zhe (Joe) Wang, Richard Zang, Xi Wang, 
and Yusaku Sako.


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


Repository: ambari


Description
---

- Navigate to Storm configs page
- Type "hook" in the property filter
- Its expected that Advanced storm-atlas-application.properties panel opens up 
by default
- Actual: its still closed. please see the screenshot attached


Diffs
-

  ambari-web/app/views/common/configs/service_configs_by_category_view.js 
3eebfea 

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


Testing
---

Verified manually.
Ambari-web unit test pass.
30374 tests complete (27 seconds)
151 tests pending


Thanks,

Vivek Ratnavel Subramanian