-
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
[16.0][IMP] add button Create Shipping Label #3
base: 16.0
Are you sure you want to change the base?
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.
hi, one requirement is missing:
Note: if on DO, carrier_id is not set --> raise a warning "Please, set a carrier."
will be useful for returns
@@ -236,3 +236,8 @@ def generate_postlogistics_shipping_labels(self, package_ids=None): | |||
"""Add label generation for PostLogistics""" | |||
self.ensure_one() | |||
return self._generate_postlogistics_label(package_ids=package_ids) | |||
|
|||
def action_generate_carrier_label(self): | |||
for pick in self: |
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.
for the sake of factorization, why not call postlogistics_send_shipping
here?
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.
hello @nilshamerlinck, this PR has been updated.
name="action_generate_carrier_label" | ||
help="Create Shipping Label 🚚" | ||
attrs="{'invisible': [('state', '!=', 'done')]}" | ||
string="Shipping Label 🚚" |
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.
string="Shipping Label 🚚" | |
string="Create Shipping Label 🚚" |
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.
hello @nilshamerlinck, this PR has been updated.
59054f6
to
4529665
Compare
4529665
to
7b8c474
Compare
Create Shipping Label
back indelivery_postlogistics
Validate stock.picking
orCreate Shipping Label
--> generate and add an attachment