-
Notifications
You must be signed in to change notification settings - Fork 23k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[FW][FIX] web: valid, then invalid, then valid field value is correct #155621
[FW][FIX] web: valid, then invalid, then valid field value is correct #155621
Conversation
@SimonGenin @aab-odoo cherrypicking of pull request #154991 failed. stdout:
stderr:
Either perform the forward-port manually (and push to this branch, proceeding as usual) or close this PR (maybe?). In the former case, you may want to edit this PR message as well. More info at https://github.com/odoo/odoo/wiki/Mergebot#forward-port |
The commit 7452f85 was fixing the problem in a simple way. It however wasn't good enough of a fix as we discovered we were triggering too many onChange after the forward port. So we revert commit 7452f85, and backport 5ffc604 from 17.1. It adds a method on the model to reset the validity of a field. It was not what felt the best, but as it is working and we just want to fix the problem in stable, we go for it. opw 3680495 X-original-commit: 35d49cc
8d00885
to
5327b6f
Compare
@robodoo r+ |
The commit 7452f85 was fixing the problem in a simple way. It however wasn't good enough of a fix as we discovered we were triggering too many onChange after the forward port. So we revert commit 7452f85, and backport 5ffc604 from 17.1. It adds a method on the model to reset the validity of a field. It was not what felt the best, but as it is working and we just want to fix the problem in stable, we go for it. opw 3680495 closes #155621 X-original-commit: 35d49cc Signed-off-by: Aaron Bohy (aab) <aab@odoo.com> Signed-off-by: Simon Genin (ges@odoo) <ges@odoo.com>
The commit odoo-dev@7452f85 was fixing the
problem in a simple way. It however wasn't good enough of a fix as we
discovered we were triggering too many onChange after the forward port.
So we revert commit odoo-dev@7452f85, and backport odoo-dev@5ffc604 from 17.1. It adds
a method on the model to reset the validity of a field.
It was not what felt the best, but as it is working and we just want to
fix the problem in stable, we go for it.
Forward-Port-Of: #155442
Forward-Port-Of: #154991