[17.0][MIG] base_delivery_carrier_label #807

David-Luis-Mora · 2024-05-06T13:44:15Z

No description provided.

…ng_label

… nesting, few typos [ADD] access rights for the shipping.label model

[IMP] remove unused pdf_list variable and rename pdf variable to label as a label can have other printing format [IMP] remove readonly on carrier_tracking_ref [IMP] remove carrier_tracking_ref redefinition as already defined in delivery/stock.py [ADD] stock users rights [ADD] file type on shipping label [ADD] pot file [ADD] fr.po

…per pack (stock.tracking) [IMP] set tracking_id as an optional key of dict returned by label generation and improve doc string [IMP] rename generate_pack_labels in generate_shipping_labels as it can also generate a single label for a picking without pack notion [IMP] set labels on stock.picking instead of stock.picking.out in order to let the creation of return labels to provide to the customer [FIX] xml view and copy what is needed on stock.picking.out and stock.picking.in as it won't be inherited from stock.picking.out

[FIX] delivery.carrier has no access rights for stock users, hence remove it for delivery.carrier.option

…by raising a orm.except_orm [FIX] trigger carrier_id on change when creating a stock.picking to set default options from sale.order [IMP] change error message of unimplemented default to a more user friendly message

…te the label only for a selection of packs in a picking the button call action_generate_carrier_label() without the 'context' keyword argument so the context ends in the tracking_ids argument. So the method now conforms to the cr, uid, ids, context signature and just forwards to a new generate_labels() method that should be used from within the code do not overwrite the option_ids on create of stock.picking if already set in values, generalize the method to all types of picking (added in/out) Set the default options of the delivery method when it is changed and the options are not set (typically a write from the code or a mass update)

docstring

[FIX] action name [IMP] add descrption field to carrier option

[IMP] file_type field inheritance in shipping.label [IMP] add field selection sort

[FIX] docstring [FIX] _customize_sender_address return method [IMP] add comment in _customize_sender_adress() [FIX] method name to get sender address [FIX] method comment

move modules in __unported__

[REF] port to v8 with old api [MIG] to v8 base_deliv_label, deliv_carrier_deposit [IMP] add parcel_tracking field [IMP] add _get_packages_from_picking() add carrier_id in tree view

[FIX] complete view

Better layout of default report for deposit slip For stock.quant.package, order by id desc to have the latest first Code cleanup Button "Create Shipping Label" should be used when the picking in Done, because in v8 the packaging is handled in the "Transfer" wizard (in v7, the packaging could be made when the picking is waiting/ready) Display number of packages on deposit slip Add weight in tree view of packages

Remove an unused file 'reports.xml' Remove the selection wrappers Use the name of the method as string instead

Move module description in README.rst

Take the destination package. If empty, the package is moved so take the source one. The options must be filtered by the selected carrier

In order to get visibility on https://www.odoo.com/apps the OCA board has decided to add the OCA as author of all the addons maintained as part of the association.

Updated by "Update PO files to match POT (msgmerge)" hook in Weblate. Translation: delivery-carrier-16.0/delivery-carrier-16.0-base_delivery_carrier_label Translate-URL: https://translation.odoo-community.org/projects/delivery-carrier-16-0/delivery-carrier-16-0-base_delivery_carrier_label/

Currently translated at 10.8% (9 of 83 strings) Translation: delivery-carrier-16.0/delivery-carrier-16.0-base_delivery_carrier_label Translate-URL: https://translation.odoo-community.org/projects/delivery-carrier-16-0/delivery-carrier-16-0-base_delivery_carrier_label/it/

Currently translated at 40.9% (34 of 83 strings) Translation: delivery-carrier-16.0/delivery-carrier-16.0-base_delivery_carrier_label Translate-URL: https://translation.odoo-community.org/projects/delivery-carrier-16-0/delivery-carrier-16-0-base_delivery_carrier_label/sl/

…received in the data from the label generation

…ef and uri

Currently translated at 100.0% (83 of 83 strings) Translation: delivery-carrier-16.0/delivery-carrier-16.0-base_delivery_carrier_label Translate-URL: https://translation.odoo-community.org/projects/delivery-carrier-16-0/delivery-carrier-16-0-base_delivery_carrier_label/es/

Updated by "Update PO files to match POT (msgmerge)" hook in Weblate. Translation: delivery-carrier-16.0/delivery-carrier-16.0-base_delivery_carrier_label Translate-URL: https://translation.odoo-community.org/projects/delivery-carrier-16-0/delivery-carrier-16-0-base_delivery_carrier_label/

Better use of logging so tools like sentry deduplicate the message.

Currently translated at 83.1% (69 of 83 strings) Translation: delivery-carrier-16.0/delivery-carrier-16.0-base_delivery_carrier_label Translate-URL: https://translation.odoo-community.org/projects/delivery-carrier-16-0/delivery-carrier-16-0-base_delivery_carrier_label/fr/

Currently translated at 10.8% (9 of 83 strings) Translation: delivery-carrier-16.0/delivery-carrier-16.0-base_delivery_carrier_label Translate-URL: https://translation.odoo-community.org/projects/delivery-carrier-16-0/delivery-carrier-16-0-base_delivery_carrier_label/it/

rousseldenis · 2024-05-08T06:58:14Z

/ocabot migration base_delivery_carrier_label

rousseldenis · 2024-05-08T06:58:55Z

@David-Luis-Mora Thanks for this. Could you check tests ?

yankinmax · 2024-06-04T16:55:18Z

@David-Luis-Mora Can you pls double check the dependency: delivery should be stock_delivery as it has been changed in v17.
And fix the views:

<record id="view_picking_withcarrier_out_form" model="ir.ui.view">
    <field name="model">stock.picking</field>
    <field name="inherit_id" ref="delivery.view_picking_withcarrier_out_form" />
    <field name="arch" type="xml">
        <xpath expr="//page//field[@name='carrier_id']" position="after">
            <field name="carrier_code" />
            <field
                name="option_ids"
                domain="[('carrier_id', '=', carrier_id)]"
                widget="many2many_tags"
                options="{'color_field': 'color', 'no_create_edit': True}"
            />
        </xpath>
    </field>
</record>

Should inherit from stock_delivery.view_picking_withcarrier_out_form
Also stock_delivery.vpicktree_view_tree
Can you pls update?

yankinmax · 2024-06-04T17:00:19Z

@David-Luis-Mora if you don't have time I can open a new PR to supersede yours and finish the job. Thanks in advance

yankinmax · 2024-06-05T08:06:01Z

@David-Luis-Mora @rousseldenis
I've opened another PR including fixes of this one to speed up migration:

[17.0][MIG] base_delivery_carrier_label: Migration to 17.0 #828

rousseldenis · 2024-06-05T09:11:28Z

@yankinmax I think we can close this as @David-Luis-Mora has no more activity on github.

yvaucher and others added 30 commits May 6, 2024 12:19

[ADD] base_delivery_carrier_label portage of Akretion delivery_shippi…

08672b6

…ng_label

[CHG] pep-8 and pep-257, return eagerly in option_ids_change to avoid…

ce2cec4

… nesting, few typos [ADD] access rights for the shipping.label model

[DEL] Remove unused views

373ba95

[FIX] load the ir.model.access.csv file

b169f30

[FIX] delivery.carrier has no access rights for stock users, hence remove it for delivery.carrier.option

[IMP] add a menu entry 'Carriers' in settings

248c7e4

base_delivery_carrier_label - Improve carrier type selection inheritance

afd4033

docstring

Launchpad automatic translations update.

3566f29

[IMP] add common view action to use by module install

31c55db

[FIX] action name [IMP] add descrption field to carrier option

When tracking_ids is defined, generate one default label per tracking id

00a6126

[IMP] complete translation and fix view

ab5e3ce

[IMP] file_type field inheritance in shipping.label [IMP] add field selection sort

Launchpad automatic translations update.

b5c4ffe

[IMP] add _customize_sender_address method

eeb2224

[FIX] docstring [FIX] _customize_sender_address return method [IMP] add comment in _customize_sender_adress() [FIX] method name to get sender address [FIX] method comment

set installable to false

46bb584

move modules in __unported__

[REF] remove stock.picking.in and .out

bc76a7e

[REF] port to v8 with old api [MIG] to v8 base_deliv_label, deliv_carrier_deposit [IMP] add parcel_tracking field [IMP] add _get_packages_from_picking() add carrier_id in tree view

[FIX] add explicitely attachment_id (previously included by orm)

8c50b8c

[FIX] complete view

[FIX] weight field type

959d2dc

Migrate to new API

3791a1f

Remove an unused file 'reports.xml' Remove the selection wrappers Use the name of the method as string instead

Remove _order on stock.quant.packages, as asked by Guewen

93f0cac

[FIX] writing package_id field in shipping.label

ec1f94b

[Usability] Better name_get on stock.quant.package

8e15c32

Move module description in README.rst

Allow to generate labels when we moved an entire package

47a2b85

Take the destination package. If empty, the package is moved so take the source one. The options must be filtered by the selected carrier

Add OCA as author of OCA addons

b39f4a1

In order to get visibility on https://www.odoo.com/apps the OCA board has decided to add the OCA as author of all the addons maintained as part of the association.

[PEP8]

9a4b097

add and generate initial german translations

de58c4f

[NEW] Added spanish translation

47d1b56

oca-ci and others added 17 commits May 6, 2024 12:19

[UPD] Update base_delivery_carrier_label.pot

e732704

[UPD] README.rst

8188510

[IMP] base_delivery_carrier_label : write the parcel tracking uri if …

20cda69

…received in the data from the label generation

[IMP] base_delivery_carrier_label : add test about package tracking r…

8ca7441

…ef and uri

base_delivery_carrier_label 16.0.1.1.0

429290d

[UPD] README.rst

72e7b62

[IMP] improve logging

0236071

Better use of logging so tools like sentry deduplicate the message.

[BOT] post-merge updates

0ca7d05

[IMP] : pre-commit auto fixes

b434da1

[MIG] base_delivery_carrier_label: Migration to 17.0

45be28c

David-Luis-Mora mentioned this pull request May 6, 2024

Migration to version 17.0 #729

Open

34 tasks

rousseldenis added needs review migration labels May 8, 2024

OCA-git-bot added this to the 17.0 milestone May 8, 2024

yankinmax mentioned this pull request Jun 4, 2024

[17.0][MIG] delivery_roulier: Migration to 17.0 #826

Merged

2 tasks

yankinmax mentioned this pull request Jun 5, 2024

[17.0][MIG] base_delivery_carrier_label: Migration to 17.0 #828

Merged

1 task

rousseldenis closed this Jun 5, 2024

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

[17.0][MIG] base_delivery_carrier_label #807

[17.0][MIG] base_delivery_carrier_label #807

David-Luis-Mora commented May 6, 2024

rousseldenis commented May 8, 2024

rousseldenis commented May 8, 2024

yankinmax commented Jun 4, 2024 •

edited

Loading

yankinmax commented Jun 4, 2024

yankinmax commented Jun 5, 2024

rousseldenis commented Jun 5, 2024

[17.0][MIG] base_delivery_carrier_label #807

[17.0][MIG] base_delivery_carrier_label #807

Conversation

David-Luis-Mora commented May 6, 2024

rousseldenis commented May 8, 2024

rousseldenis commented May 8, 2024

yankinmax commented Jun 4, 2024 • edited Loading

yankinmax commented Jun 4, 2024

yankinmax commented Jun 5, 2024

rousseldenis commented Jun 5, 2024

yankinmax commented Jun 4, 2024 •

edited

Loading