Gilad Chaplik has posted comments on this change.
Change subject: webadmin: Adding force option to remove host
......................................................................
Patch Set 1: (2 inline comments)
....................................................
File
frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/CommonApplicationConstants.java
Line 199: @DefaultStringValue("Approve operation")
Line 200: String approveOperation();
Line 201:
Line 202: @DefaultStringValue("Remove Forcefully")
Line 203: String removeForcefully();
I think the terminologhy in other places is 'Force Remove',
can you plz check it out?
Line 204:
Line 205: // General
Line 206: @DefaultStringValue("Yes")
Line 207: String yes();
....................................................
File
frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/hosts/HostListModel.java
Line 905: setWindow(model);
Line 906:
model.setTitle(ConstantsManager.getInstance().getConstants().removeHostsTitle());
Line 907: model.setHashName("remove_host"); //$NON-NLS-1$
Line 908:
model.setMessage(ConstantsManager.getInstance().getConstants().hostsMsg());
Line 909:
nice!!!
Line 910: Set<Guid> clusters = new HashSet<Guid>();
Line 911: ArrayList<String> list = new ArrayList<String>();
Line 912: for (VDS item : Linq.<VDS> Cast(getSelectedItems()))
Line 913: {
--
To view, visit http://gerrit.ovirt.org/10180
To unsubscribe, visit http://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: I9fe3814815acff8a5f230c1a3d6ebcb6e3bbc633
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Kanagaraj M <[email protected]>
Gerrit-Reviewer: Gilad Chaplik <[email protected]>
Gerrit-Reviewer: Shireesh Anjal <[email protected]>
_______________________________________________
Engine-patches mailing list
[email protected]
http://lists.ovirt.org/mailman/listinfo/engine-patches
