[node-patches] Change in ovirt-node[master]: ui: Add security check to ConfirmedEntry

fabiand at fedoraproject.org fabiand at fedoraproject.org
Fri Sep 13 07:12:50 UTC 2013


Fabian Deutsch has submitted this change and it was merged.

Change subject: ui: Add security check to ConfirmedEntry
......................................................................


ui: Add security check to ConfirmedEntry

Previously it was cumbersome to get the effects of incorrect/unsave
passwords on the UI straight. By pulling the password check code into
the ConfirmedEntry widget and adding a Notice field, the whole process
is simpliefied from a "user" perspektive (plugin writer).

Change-Id: I302ec06290d6c1dd26c7b4e8b3c13727ec8b1131
Signed-off-by: Fabian Deutsch <fabiand at fedoraproject.org>
---
M src/ovirt/node/app.py
M src/ovirt/node/ui/__init__.py
M src/ovirt/node/utils/security.py
3 files changed, 51 insertions(+), 9 deletions(-)

Approvals:
  Fabian Deutsch: Verified; Looks good to me, approved



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

Gerrit-MessageType: merged
Gerrit-Change-Id: I302ec06290d6c1dd26c7b4e8b3c13727ec8b1131
Gerrit-PatchSet: 13
Gerrit-Project: ovirt-node
Gerrit-Branch: master
Gerrit-Owner: Fabian Deutsch <fabiand at fedoraproject.org>
Gerrit-Reviewer: Fabian Deutsch <fabiand at fedoraproject.org>
Gerrit-Reviewer: Ryan Barry <rbarry at redhat.com>
Gerrit-Reviewer: oVirt Jenkins CI Server



More information about the node-patches mailing list