Eli Mesika has posted comments on this change.

Change subject: core: Add error messages for new commands.
......................................................................


Patch Set 1: (1 inline comment)

....................................................
File backend/manager/modules/dal/src/main/resources/bundles/AppErrors.properties
Line 906: VAR__ACTION__END=$action End
Line 907: VAR__ACTION__CLEAR=$action Clear
Line 908: ACTION_TYPE_EMPTY_DESCRIPTION=Cannot ${action} ${type}. Description 
can not be empty.
Line 909: ACTION_TYPE_NOT_EXTERNAL=Cannot ${action} ${type}. External steps can 
be added only to external jobs or steps.
Line 910: ACTION_TYPE_NO_PARENT=Cannot ${action} ${type}. Step should be a 
child of the Job or other Step.
I think syntax is OK the 'of' referencing both parts
Line 911: ACTION_TYPE_NO_JOB=Cannot ${action} ${type}. Job not found.
Line 912: ACTION_TYPE_NO_STEP=Cannot ${action} ${type}. Step not found.
Line 913: 
Line 914: # Register disk error


-- 
To view, visit http://gerrit.ovirt.org/16158
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I1c7a5da46f9202a4a84f54b9bc6c9e2afcde81e4
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Eli Mesika <[email protected]>
Gerrit-Reviewer: Eli Mesika <[email protected]>
Gerrit-Reviewer: Yair Zaslavsky <[email protected]>
Gerrit-Reviewer: oVirt Jenkins CI Server
_______________________________________________
Engine-patches mailing list
[email protected]
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to