Change in ovirt-engine[ovirt-engine-4.1]: webadmin: disable unsetting initally assigned network role
Code Review
gerrit at ovirt.org
Sun Feb 12 16:39:38 UTC 2017
>From Tal Nisan <tnisan at redhat.com>:
Tal Nisan has submitted this change and it was merged.
Change subject: webadmin: disable unsetting initally assigned network role
......................................................................
webadmin: disable unsetting initally assigned network role
Make a network role checkbox not editable in manage network dialog
if that was initially set and the flow is Networks->Clusters.
Change-Id: I5983a8240a8f115c11d6ed5ee2d92d50922c934b
Bug-Url: https://bugzilla.redhat.com/1329893
Signed-off-by: Yevgeny Zaspitsky <yzaspits at redhat.com>
(cherry picked from commit cd46f7b5767e403a03db155f1ada3ad65138d65a)
---
M frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/widget/table/column/AbstractCheckboxColumn.java
M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/ApplicationConstants.java
M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/popup/cluster/ClusterManageNetworkPopupView.java
M frontend/webadmin/modules/webadmin/src/main/resources/org/ovirt/engine/ui/webadmin/ApplicationConstants.properties
4 files changed, 62 insertions(+), 13 deletions(-)
Approvals:
Alona Kaplan: Looks good to me, approved
Jenkins CI: Passed CI tests
Yevgeny Zaspitsky: Verified
--
To view, visit https://gerrit.ovirt.org/72115
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I5983a8240a8f115c11d6ed5ee2d92d50922c934b
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-engine
Gerrit-Branch: ovirt-engine-4.1
Gerrit-Owner: Alona Kaplan <alkaplan at redhat.com>
Gerrit-Reviewer: Alona Kaplan <alkaplan at redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Tal Nisan <tnisan at redhat.com>
Gerrit-Reviewer: Yevgeny Zaspitsky <yzaspits at redhat.com>
Gerrit-Reviewer: gerrit-hooks <automation at ovirt.org>
More information about the Engine-commits
mailing list