Moti Asayag has posted comments on this change.

Change subject: ui: Adding fields validation
......................................................................


Patch Set 2:

Please add the same validation on the backend as well (for the sake of users 
which intend to use the api)

although external providers aren't supported from the UI, it makes sense to 
have this validation there as well, to prevent falling into the same pitfalls 
once api support is added.

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

Gerrit-MessageType: comment
Gerrit-Change-Id: Icfc3e500ad06728fc8c410baa13405633fc90da2
Gerrit-PatchSet: 2
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Yaniv Bronhaim <[email protected]>
Gerrit-Reviewer: Moti Asayag <[email protected]>
Gerrit-Reviewer: Oved Ourfali <[email protected]>
Gerrit-Reviewer: Yaniv Bronhaim <[email protected]>
Gerrit-Reviewer: [email protected]
Gerrit-Reviewer: oVirt Jenkins CI Server
Gerrit-HasComments: No
_______________________________________________
Engine-patches mailing list
[email protected]
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to