Tomas Jelinek has posted comments on this change.

Change subject: webadmin: Network builders
......................................................................


Patch Set 4: I would prefer that you didn't submit this

Of course we can compromise on some solution - I have no desire to push my one 
if not appropriate.

The problem with common super class is that the first common one is Model and 
it is far to high to this kind of logic.

I will rewrite this code to a behavior class.

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

Gerrit-MessageType: comment
Gerrit-Change-Id: Icae2963687ad7978c3caa026e7062e7a1937d320
Gerrit-PatchSet: 4
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Tomas Jelinek <[email protected]>
Gerrit-Reviewer: Alona Kaplan <[email protected]>
Gerrit-Reviewer: Daniel Erez <[email protected]>
Gerrit-Reviewer: Tomas Jelinek <[email protected]>
_______________________________________________
Engine-patches mailing list
[email protected]
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to