Bug 1693743
| Summary: | [RFE] Vertical menu fixes and other enhancements for v2v UI | ||
|---|---|---|---|
| Product: | Red Hat CloudForms Management Engine | Reporter: | Satoe Imaishi <simaishi> |
| Component: | V2V | Assignee: | Brett Thurber <bthurber> |
| Status: | CLOSED ERRATA | QA Contact: | Yadnyawalk Tale <ytale> |
| Severity: | low | Docs Contact: | Red Hat CloudForms Documentation <cloudforms-docs> |
| Priority: | medium | ||
| Version: | 5.10.1 | CC: | bthurber, dmetzger, fdupont, mturley, mzazrivec |
| Target Milestone: | GA | Keywords: | FutureFeature, ZStream |
| Target Release: | 5.10.3 | ||
| Hardware: | Unspecified | ||
| OS: | Unspecified | ||
| Whiteboard: | v2v | ||
| Fixed In Version: | 5.10.3.1 | Doc Type: | Enhancement |
| Doc Text: | Story Points: | --- | |
| Clone Of: | 1666359 | Environment: | |
| Last Closed: | 2019-04-23 07:46:25 UTC | Type: | --- |
| Regression: | --- | Mount Type: | --- |
| Documentation: | --- | CRM: | |
| Verified Versions: | Category: | --- | |
| oVirt Team: | --- | RHEL 7.3 requirements from Atomic Host: | |
| Cloudforms Team: | V2V | Target Upstream Version: | |
| Embargoed: | |||
| Bug Depends On: | 1666359 | ||
| Bug Blocks: | |||
|
Comment 2
Satoe Imaishi
2019-03-29 17:02:37 UTC
Milan, can you determine if those manageiq-ui-classic PRs are safe to backport to hammer? Yes, those two ui-classic PRs should be safe to backport. Should the cherry-picks fail, I'll create separate PRs just for hammer. New commits detected on ManageIQ/manageiq-ui-classic/hammer: https://github.com/ManageIQ/manageiq-ui-classic/commit/8d7c5f9d1f732d4e2d9fb50b9ec57d32685da25e commit 8d7c5f9d1f732d4e2d9fb50b9ec57d32685da25e Author: Martin Hradil <himdel> AuthorDate: Fri Dec 7 10:58:22 2018 -0500 Commit: Martin Hradil <himdel> CommitDate: Fri Dec 7 10:58:22 2018 -0500 Merge pull request #5063 from mzazrivec/add_menu_item_ids_to_vertical_menu Add IDs to vertical menu leaf items (cherry picked from commit 9cb9227bc7744fb071b765089d2db2e9491bf59f) https://bugzilla.redhat.com/show_bug.cgi?id=1693743 app/views/layouts/_vertical_navbar.html.haml | 6 +- 1 file changed, 3 insertions(+), 3 deletions(-) https://github.com/ManageIQ/manageiq-ui-classic/commit/20a4d4a9b47081446b8a2626831ffc4376fce061 commit 20a4d4a9b47081446b8a2626831ffc4376fce061 Author: Martin Hradil <himdel> AuthorDate: Fri Dec 14 10:05:57 2018 -0500 Commit: Martin Hradil <himdel> CommitDate: Fri Dec 14 10:05:57 2018 -0500 Merge pull request #4999 from mzazrivec/add_menu_update_mixin Add MenuUpdateMixin (cherry picked from commit f75dffbbd1b992d704196890065dd0c9af0b6098) https://bugzilla.redhat.com/show_bug.cgi?id=1693743 app/controllers/mixins/menu_update_mixin.rb | 11 + spec/controllers/mixins/menu_update_mixin_spec.rb | 21 + 2 files changed, 32 insertions(+) New commits detected on ManageIQ/manageiq-v2v/hammer: https://github.com/ManageIQ/manageiq-v2v/commit/f295e552246d656a3b4e1020c0356ab58fd583db commit f295e552246d656a3b4e1020c0356ab58fd583db Author: Michael Ro <mikerodev> AuthorDate: Mon Jan 7 12:31:51 2019 -0500 Commit: Michael Ro <mikerodev> CommitDate: Mon Jan 7 12:31:51 2019 -0500 Merge pull request #839 from mturley/prevent-settings-destroy [#802] Prevent Settings form from being destroyed on unmount (cherry picked from commit 4aad87810b1881ca53821bcc0e5812d6bf0d6848) https://bugzilla.redhat.com/show_bug.cgi?id=1693743 app/javascript/react/screens/App/Settings/index.js | 3 +- 1 file changed, 2 insertions(+), 1 deletion(-) https://github.com/ManageIQ/manageiq-v2v/commit/fff0084d4db70a595d9e3df4e1f6c1637c36b1e8 commit fff0084d4db70a595d9e3df4e1f6c1637c36b1e8 Author: Mike Turley <mike.turley.umass.edu> AuthorDate: Mon Jan 7 13:40:46 2019 -0500 Commit: Mike Turley <mike.turley.umass.edu> CommitDate: Mon Jan 7 13:40:46 2019 -0500 Merge pull request #822 from mzazrivec/phase_2_of_vertical_menu_fixes Phase 2 of vertical menu fixes (cherry picked from commit 40a7a5f41a049b0bdef3e60a12d5f691810f193f) https://bugzilla.redhat.com/show_bug.cgi?id=1693743 app/controllers/migration_controller.rb | 11 +- app/javascript/common/menu.js | 25 + app/javascript/react/config/Routes.js | 11 +- app/javascript/react/config/config.js | 24 +- app/javascript/react/index.js | 18 +- app/javascript/react/screens/App/Mappings/Mappings.js | 2 +- app/javascript/react/screens/App/Mappings/screens/MappingWizard/components/MappingWizardResultsStep/MappingWizardResultsStep.js | 2 +- app/javascript/react/screens/App/Overview/Overview.js | 4 +- app/javascript/react/screens/App/Overview/components/Migrations/InProgressWithDetailCard.js | 2 +- app/javascript/react/screens/App/Overview/components/Migrations/MigrationsCompletedList.js | 4 +- app/javascript/react/screens/App/Overview/components/Migrations/MigrationsNotStartedList.js | 4 +- app/javascript/react/screens/App/Overview/components/Migrations/__test__/MigrationsNotStartedList.test.js | 2 +- app/javascript/react/screens/App/Plan/Plan.js | 3 +- app/javascript/react/screens/App/Settings/Settings.js | 2 +- app/javascript/react/screens/App/Settings/__tests__/__snapshots__/Settings.test.js.snap | 2 +- app/javascript/react/screens/App/common/NotificationList/NotificationList.js | 3 +- app/javascript/redux/actions/routerActions.js | 9 +- config/routes.rb | 9 +- lib/manageiq/v2v/engine.rb | 6 +- 19 files changed, 97 insertions(+), 46 deletions(-) Checked routing with browsing history Checked correct vertical menu item is getting highlighted Checked if Compute ➛ Migrate takes you to previously visited url Verified on: 5.10.3.1.20190408182335_d0fa1a0 Since the problem described in this bug report should be resolved in a recent advisory, it has been closed with a resolution of ERRATA. For information on the advisory, and where to find the updated files, follow the link below. If the solution does not work for you, open a new bug report. https://access.redhat.com/errata/RHSA-2019:0796 |