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
[12.0] [FIX] stock_analytic: Analytic account field is in wrong page #258
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍 LGTM
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think there should be two xpaths instead. One for the move_line_ids_without_package
and the other for the move_ids_without_package
.
@mreficent The problem is, the field is readonly in In fact, correct me if I'm wrong, but the fields used to have different names in 11.0 so could it be that the inherited view used the wrong field in the migration from 11.0 to 12.0? If so, this PR really is a fix to that problem and we could make a separate issue of adding the analytic account field to |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ok
/ocabot merge patch |
Hey, thanks for contributing! Proceeding to merge this for you. |
Congratulations, your PR was merged at c4b6706. Thanks a lot for contributing to OCA. ❤️ |
Fixes #257