Bug 1624248
Summary: | Bad UI after adding a schedule for report | |||
---|---|---|---|---|
Product: | Red Hat CloudForms Management Engine | Reporter: | Parthvi Vala <pvala> | |
Component: | UI - OPS | Assignee: | Harpreet Kataria <hkataria> | |
Status: | CLOSED CURRENTRELEASE | QA Contact: | Parthvi Vala <pvala> | |
Severity: | high | Docs Contact: | ||
Priority: | high | |||
Version: | 5.8.5 | CC: | dmetzger, hkataria, lavenel, mpovolny, obarenbo, simaishi | |
Target Milestone: | GA | Keywords: | TestOnly, ZStream | |
Target Release: | 5.10.0 | |||
Hardware: | Unspecified | |||
OS: | Unspecified | |||
Whiteboard: | ||||
Fixed In Version: | 5.10.0.15 | Doc Type: | If docs needed, set a value | |
Doc Text: | Story Points: | --- | ||
Clone Of: | ||||
: | 1646564 (view as bug list) | Environment: | ||
Last Closed: | 2019-02-12 16:50:41 UTC | Type: | Bug | |
Regression: | --- | Mount Type: | --- | |
Documentation: | --- | CRM: | ||
Verified Versions: | Category: | Bug | ||
oVirt Team: | --- | RHEL 7.3 requirements from Atomic Host: | ||
Cloudforms Team: | --- | Target Upstream Version: | ||
Embargoed: | ||||
Bug Depends On: | ||||
Bug Blocks: | 1646564 |
Description
Parthvi Vala
2018-08-31 06:14:11 UTC
New commit detected on ManageIQ/manageiq-ui-classic/master: https://github.com/ManageIQ/manageiq-ui-classic/commit/62aec06c04997c34b682f95f21e08ca9ce10efe2 commit 62aec06c04997c34b682f95f21e08ca9ce10efe2 Author: Harpreet Kataria <hkataria> AuthorDate: Thu Sep 6 11:28:09 2018 -0400 Commit: Harpreet Kataria <hkataria> CommitDate: Thu Sep 6 11:28:09 2018 -0400 Fixed Schedules accordion swapping logic. Only need to set params[:accord] when add button is pressed from Reports accordion, setting of acord parameter causes acoridon to be swapped to Schedule accordion by Explorer presenter code. Undid code changes made in https://github.com/ManageIQ/manageiq-ui-classic/pull/3399 this was causing issues when adding a schedule from Reports accordion. Also need to hide form buttons post accordion switch when schedule is added from Reports accordion. Fixes https://bugzilla.redhat.com/show_bug.cgi?id=1624248 app/assets/javascripts/miq_explorer.js | 3 +- app/controllers/report_controller.rb | 1 + app/controllers/report_controller/schedules.rb | 3 +- spec/controllers/report_controller/schedules_spec.rb | 31 + 4 files changed, 34 insertions(+), 4 deletions(-) Verified on 5.10.0.15.20180914163601_fb02d7e. It works perfectly. Great work! *** Bug 1636744 has been marked as a duplicate of this bug. *** |