-
-
Notifications
You must be signed in to change notification settings - Fork 720
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
[14.0][FIX] product_route_profile fix view #1939
Conversation
Hi @Kev-Roche, |
Thanks! Can you amend your commit to something like
(is how I'm cherrypicking this into the 17.0 migration at #2097) |
@bguillot Could you attend @StefanRijnhart comments? Thanks |
Set route_ids as readonly to avoid triggering inverse method and changing the route_profile_id when adding force_route_profile_id
d506d7b
to
042d190
Compare
sorry, I missed the comments. It's done |
This PR has the |
/ocabot merge patch |
On my way to merge this fine PR! |
Congratulations, your PR was merged at 451baec. Thanks a lot for contributing to OCA. ❤️ |
Hello ,
the route_ids field must be read-only in the view to avoid triggering the inverse method, which recalculates the route_profile_id field when the force_route_profile_id field is filled in.