Bug 1481637 - Inconsistency between filled name and name in accordion of Provision Dialogs
Summary: Inconsistency between filled name and name in accordion of Provision Dialogs
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: Red Hat CloudForms Management Engine
Classification: Red Hat
Component: UI - OPS
Version: 5.8.0
Hardware: Unspecified
OS: Unspecified
medium
medium
Target Milestone: GA
: 5.9.0
Assignee: Greg Piatigorski
QA Contact: Yadnyawalk Tale
URL:
Whiteboard: ui:automate:dialog
Depends On:
Blocks: 1512694
TreeView+ depends on / blocked
 
Reported: 2017-08-15 09:34 UTC by Yadnyawalk Tale
Modified: 2018-03-06 14:36 UTC (History)
5 users (show)

Fixed In Version: 5.9.0.1
Doc Type: If docs needed, set a value
Doc Text:
Clone Of:
: 1512694 (view as bug list)
Environment:
Last Closed: 2018-03-06 14:36:33 UTC
Category: Bug
Cloudforms Team: CFME Core
Target Upstream Version:
Embargoed:
ytale: automate_bug-


Attachments (Terms of Use)
working_names (196.83 KB, image/jpeg)
2017-11-27 06:44 UTC, Yadnyawalk Tale
no flags Details

Description Yadnyawalk Tale 2017-08-15 09:34:04 UTC
Description of problem:
Inconsistency between filled dialog name and added dialog name in accordion of Provision Dialogs

Version-Release number of selected component:
5.8.1.5.20170725160636_e433fc0

How reproducible:
100% (Always)

Steps to Reproduce:
1. Navigate to Automation -> Automate -> Customization
2. Add new dialog there (Put name as 'dialog_name' and 'description as dialog_desc'

Actual results:
Showing dialog_desc as a added dialog name in accordion

Expected results:
Should display dialog_name instead of dialog_desc

Additional info: 
Linked with github issue -> https://github.com/ManageIQ/manageiq/issues/15742

Comment 4 Greg Piatigorski 2017-08-28 18:46:21 UTC
NOTE: 
Code refactored to display dialog Add/Save/Cancel (edit) confirmation messages with Dialog Descriptions (to match other UI areas for consistency).

Leaving accordion tree node to display dialog descriptions, rather than names, due to dialog names possibly being too short/cut off and not be descriptive enough as tree node for quick navigation.

Comment 6 CFME Bot 2017-09-20 16:53:51 UTC
New commit detected on ManageIQ/manageiq-ui-classic/master:
https://github.com/ManageIQ/manageiq-ui-classic/commit/a20a4b927840bf4391df93e6bf49b09a8edfa9ed

commit a20a4b927840bf4391df93e6bf49b09a8edfa9ed
Author:     Greg Piatigorski <gpiatigo>
AuthorDate: Mon Aug 28 12:57:11 2017 -0700
Commit:     Greg Piatigorski <gpiatigo>
CommitDate: Mon Aug 28 13:19:17 2017 -0700

    Fix custom Provisioning dialogs confirmation flash messages
    
    https://bugzilla.redhat.com/show_bug.cgi?id=1481637

 app/controllers/miq_ae_customization_controller/old_dialogs.rb | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

Comment 8 Yadnyawalk Tale 2017-11-27 06:44:01 UTC
Created attachment 1359302 [details]
working_names

Hey,

Showing 'desc' added as well as deleted, working fine with this fix!
Verified on: 5.9.0.10.20171121225911_a2209d2 

Thanks!


Note You need to log in before you can comment on or make changes to this bug.