Merge lp:~canonical-isd-hackers/canonical-identity-provider/bug_579920 into lp:canonical-identity-provider/release
Proposed by
David Owen
Status: | Merged |
---|---|
Merged at revision: | 47 |
Proposed branch: | lp:~canonical-isd-hackers/canonical-identity-provider/bug_579920 |
Merge into: | lp:canonical-identity-provider/release |
Diff against target: |
41 lines (+21/-0) 2 files modified
identityprovider/forms.py (+1/-0) identityprovider/tests/test_forms.py (+20/-0) |
To merge this branch: | bzr merge lp:~canonical-isd-hackers/canonical-identity-provider/bug_579920 |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Danny Tamez | Pending | ||
Review via email:
|
This proposal supersedes a proposal from 2010-05-20.
Commit message
If the first password field has errors, errors regarding the second password field are suppressed (whether or not it matches the first field).
Includes unit tests.
To post a comment you must log in.
First bug fix is looking good David. Please add a test and I'll approve. Also,even once approved please do not merge into trunk until we're out of code freeze.