Bug 1570152
| Summary: | Dynamic check box does not update in Classic UI | ||||||||
|---|---|---|---|---|---|---|---|---|---|
| Product: | Red Hat CloudForms Management Engine | Reporter: | Shveta <sshveta> | ||||||
| Component: | Automate | Assignee: | Tina Fitzgerald <tfitzger> | ||||||
| Status: | CLOSED CURRENTRELEASE | QA Contact: | Niyaz Akhtar Ansari <nansari> | ||||||
| Severity: | high | Docs Contact: | |||||||
| Priority: | high | ||||||||
| Version: | 5.9.0 | CC: | cpelland, dmetzger, eclarizi, gmccullo, greartes, hkataria, lavenel, mkanoor, mpovolny, nansari, obarenbo, rspagnol, tfitzger, wfitzger, ytale | ||||||
| Target Milestone: | GA | Keywords: | Regression, TestOnly, ZStream | ||||||
| Target Release: | 5.10.0 | ||||||||
| Hardware: | Unspecified | ||||||||
| OS: | Unspecified | ||||||||
| Whiteboard: | |||||||||
| Fixed In Version: | 5.10.0.0 | Doc Type: | If docs needed, set a value | ||||||
| Doc Text: | Story Points: | --- | |||||||
| Clone Of: | |||||||||
| : | 1571976 (view as bug list) | Environment: | |||||||
| Last Closed: | 2019-02-11 14:01:58 UTC | Type: | Bug | ||||||
| Regression: | --- | Mount Type: | --- | ||||||
| Documentation: | --- | CRM: | |||||||
| Verified Versions: | Category: | --- | |||||||
| oVirt Team: | --- | RHEL 7.3 requirements from Atomic Host: | |||||||
| Cloudforms Team: | CFME Core | Target Upstream Version: | |||||||
| Embargoed: | |||||||||
| Bug Depends On: | |||||||||
| Bug Blocks: | 1571976 | ||||||||
| Attachments: |
|
||||||||
|
Description
Shveta
2018-04-20 18:16:08 UTC
Created attachment 1424644 [details]
Automate method
This is also a bug in SUI So I can help shortcut this for someone on the Classic UI team. This issue was with ui-components repo and was solved already. The SUI currently has the fix applied for this version of release but because of how the SUI uses NPM for dependency updates and Classic UI uses bower, I suspect when it was fixed and version bumped in SUI, it might not have gotten version bumped in bower and integrated and retested in Classic UI. This should be a fairly simple fix for someone to lookup and apply a newer version on Classic UI. Hit submit too soon last time. Erik - I believe you might have a work-around for this issue. Please add details here. We will need a couple of more GH pull requests after this one gets merged. SUI and Classic UI need PR's created after this fix is released to take advantage of the fix. New commit detected on ManageIQ/ui-components/master: https://github.com/ManageIQ/ui-components/commit/454b998b06fef24291fb3a3581385d1a6d8a3a6e commit 454b998b06fef24291fb3a3581385d1a6d8a3a6e Author: Erik Clarizio <eclarizio> AuthorDate: Mon Apr 23 14:28:26 2018 -0400 Commit: Erik Clarizio <eclarizio> CommitDate: Mon Apr 23 14:28:26 2018 -0400 Use updated vaules as default for checkbox when field is dynamic https://bugzilla.redhat.com/show_bug.cgi?id=1570152 src/dialog-user/services/dialogData.spec.ts | 43 +- src/dialog-user/services/dialogData.ts | 9 + 2 files changed, 42 insertions(+), 10 deletions(-) In response to Greg and clearing the needinfo: I was able to fix the issue on the reproducing appliance by removing the default value for the checkbox in the database. Alternatively I suppose you could export the dialog, manually edit the yaml to not have a default value for the checkbox field in question, and then re-import. GH PR for Service UI to take advantage of this bugfix on G release. https://github.com/ManageIQ/manageiq-ui-service/pull/1426 GH PR for Classic UI to take advantage of the bugfix on G release. https://github.com/ManageIQ/manageiq-ui-classic/pull/3845 Waiting for new build Works on 5.9.2.4 SUI Verified in Version 5.10.0.14.20180831154906_2f9a1dd |