-
-
Notifications
You must be signed in to change notification settings - Fork 230
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
[16.0][FIX] stock_picking_report_valued: Use original qty done column from … #308
[16.0][FIX] stock_picking_report_valued: Use original qty done column from … #308
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.
Make sense! It should be done with good inheritance
👍🏼
@@ -94,36 +88,18 @@ | |||
inherit_id="stock.stock_report_delivery_has_serial_move_line" | |||
> | |||
<xpath expr="//td[@name='move_line_lot_qty_done']" position="after"> |
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.
suggestion: if possible, it's better to add a name
attribute to the th
above and find the corresponding position here in the tbody using this trick. Otherwise, other modules that extend the same report could put some column in the middle and break the layout.
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.
Cool trick! I save it for the future. Thanks.
Now I fix with name
for this case.
…Odoo I don't understand why it is necessary to hide the original qty done column. This makes other modules that modify the qty done column not work. @moduon MT-5825
68c12ae
to
8868129
Compare
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
/ocabot merge minor |
On my way to merge this fine PR! |
Congratulations, your PR was merged at 94ce82f. Thanks a lot for contributing to OCA. ❤️ |
I don't understand why it is necessary to hide the original qty done column. This makes other modules that inherit the qty done column not work.
@Shide @yajo please review.
@moduon MT-5825