Merge lp:~therp-nl/therp-backports/web-6.1-lp1013636-one2many_fields_honour_requiredness into lp:therp-backports/web-6.1
Proposed by
Stefan Rijnhart (Opener)
Status: | Merged | ||||
---|---|---|---|---|---|
Merged at revision: | 2504 | ||||
Proposed branch: | lp:~therp-nl/therp-backports/web-6.1-lp1013636-one2many_fields_honour_requiredness | ||||
Merge into: | lp:therp-backports/web-6.1 | ||||
Diff against target: |
13 lines (+3/-0) 1 file modified
addons/web/static/src/js/view_form.js (+3/-0) |
||||
To merge this branch: | bzr merge lp:~therp-nl/therp-backports/web-6.1-lp1013636-one2many_fields_honour_requiredness | ||||
Related bugs: |
|
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Holger Brunn (Therp) | Approve | ||
Review via email: mp+156301@code.launchpad.net |
To post a comment you must log in.
For the other widgets, I see them touching this.invalid mostly in the function validate(), wheras is_valid returns false when the widget is invalid.
I think you either should move your code to validate() or just return false in your case.