-
Notifications
You must be signed in to change notification settings - Fork 0
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 stock picking report cmr #1
base: 14.0
Are you sure you want to change the base?
Conversation
initial commit This module allows the printing of the CMR report from a given delivery note.
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.
@ntsirintanis AFAICS we're missing package_type
for item 8 in the report, where we need to render the display name of the selection field.
Otherwise, this seems to go well, but @thomaspaulb should also review
<t | ||
t-if="o.shipping_weight" | ||
> | ||
<div | ||
class="text-bigger text-dark mt-1 ml-2" | ||
> | ||
<span | ||
t-field="o.shipping_weight" | ||
/> |
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.
@ntsirintanis make it o.weight
rather - I don't see shipping_weight
being filled, on second viewing.
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.
shipping_weight is basically computed from package weight. Have you checked if the weight field in packages if filled?
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.
class="text-bigger text-dark mt-1 ml-2" | ||
> | ||
<span | ||
t-esc="', '.join(o.move_line_ids.mapped('result_package_id.packaging_id.name'))" |
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.
@gjotten this is basically package_type
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.
Hmm that's some ways away from strictly package_type
on the picking - not sure about this one either, could again be something to do with their setup; @thomaspaulb
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.
ah I see now. stock.picking.package_type appears to be a customer specific field
No description provided.