Skip to content
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

[MIG] delivery_seur: Migration to 14.0 #2290

Closed
wants to merge 37 commits into from

Conversation

Vicenteagf
Copy link

Migración a 14.0
Dependencia delivery_price_method

rlizana and others added 30 commits May 16, 2022 12:03
[FIX] rename views to  && [FIX] small fixes

[UPD] remove depend to delivery_price_method

[UDP] change depend stock_picking_delivery_info_computation to delivery_package_number

[FIX] work with latin characters

[UPD] test replace to SavepointCase

[UPD] test replace to setUpClass

[UPD] force carrier margin to 0

[UPD] test create picelist

[REM] set a margin value

[UPD] .rst files

[UPD] README.rst
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate.

Translation: l10n-spain-12.0/l10n-spain-12.0-delivery_seur
Translate-URL: https://translation.odoo-community.org/projects/l10n-spain-12-0/l10n-spain-12-0-delivery_seur/
… expedition

[UDP] delivery_seur: information for create shipping in 'prepare' method for ease overwrite

[UPD] delivery_seur: add comment

[UPD] comments
delivery_seur 12.0.1.0.1
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate.

Translation: l10n-spain-12.0/l10n-spain-12.0-delivery_seur
Translate-URL: https://translation.odoo-community.org/projects/l10n-spain-12-0/l10n-spain-12-0-delivery_seur/
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate.

Translation: l10n-spain-12.0/l10n-spain-12.0-delivery_seur
Translate-URL: https://translation.odoo-community.org/projects/l10n-spain-12-0/l10n-spain-12-0-delivery_seur/
- Currently there's no support for Cash on Delivery expeditions. We
shouldn't give any value to the Cash on Delivery field as it's recorded
in the expedition and can lead to errors.

delivery_seur 12.0.1.0.2
According to SEUR, the value expected is the weight by package not the
total of the expedition.
- Add easy traceability of communications to SEUR with last sent and
received data.
- Better exception handling as in some case we didn't have a different
error response.
Follow the same format as in the delivery slip

[UPD] Update delivery_seur.pot

delivery_seur 12.0.1.1.1

Update translation files

Updated by "Update PO files to match POT (msgmerge)" hook in Weblate.

Translation: l10n-spain-12.0/l10n-spain-12.0-delivery_seur
Translate-URL: https://translation.odoo-community.org/projects/l10n-spain-12-0/l10n-spain-12-0-delivery_seur/
Non implemented methods should raise informative exceptions if used

[UPD] Update delivery_seur.pot

delivery_seur 12.0.1.1.2

Update translation files

Updated by "Update PO files to match POT (msgmerge)" hook in Weblate.

Translation: l10n-spain-12.0/l10n-spain-12.0-delivery_seur
Translate-URL: https://translation.odoo-community.org/projects/l10n-spain-12-0/l10n-spain-12-0-delivery_seur/
SEUR sends the Zebra label with an CI10 command, wich stands for spanish
character set encoding. We must encode the resultant txt file with
iso-8859-15 as well to support accents properly.

[UPD] Update delivery_seur.pot

delivery_seur 12.0.1.1.3

Update translation files

Updated by "Update PO files to match POT (msgmerge)" hook in Weblate.

Translation: l10n-spain-12.0/l10n-spain-12.0-delivery_seur
Translate-URL: https://translation.odoo-community.org/projects/l10n-spain-12-0/l10n-spain-12-0-delivery_seur/
Although it will raise another exception, at least it will show the
proper message and not a traceback.

Fixes OCA#1605
Currently translated at 100.0% (71 of 71 strings)

Translation: l10n-spain-13.0/l10n-spain-13.0-delivery_seur
Translate-URL: https://translation.odoo-community.org/projects/l10n-spain-13-0/l10n-spain-13-0-delivery_seur/es/
Currently translated at 100.0% (68 of 68 strings)

Translation: l10n-spain-13.0/l10n-spain-13.0-delivery_seur
Translate-URL: https://translation.odoo-community.org/projects/l10n-spain-13-0/l10n-spain-13-0-delivery_seur/ca/
OCA-git-bot and others added 7 commits May 16, 2022 12:03
If we provide the display name of the partner, there's a high chance
that it's trimmed in the label leading to confusion.

This way, we can keep the relevant data considering if the contact is a
delivery address, a child contact or a commercial partner.

TT33632
In v12, with the parameter datas_fname we could force the file to be
downloaded as such filename. Now that parameter doesn't exist and the
filename format is lost from the previous version.

odoo/odoo@c212cfe

This is specilly problematic with txt format and some printer which
doesnt support joliet filenames or even files without extension.

The mimetype for text files was also wrong, as 'application/txt' doesn't
exist. See Mozilla Dev docs for more info on this:

https://developer.mozilla.org/en-US/docs/Web/HTTP/Basics_of_HTTP/MIME_types/Common_types

TT34599
@victoralmau
Copy link
Member

delivery_price_method is already merged since OCA/delivery-carrier#518
Do you plan to continue with this PR?

@victoralmau
Copy link
Member

Please, include #2373 changes

Copy link
Member

@victoralmau victoralmau left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Será necesario hacer también lo siguiente:

@Vicenteagf Vicenteagf force-pushed the 14.0-mig-delivery_seur branch from a50e1e7 to c7ebe19 Compare August 17, 2022 14:43
@Vicenteagf
Copy link
Author

Será necesario hacer también lo siguiente:

* Mantener el historial de commits (#2373) en el orden en cuestión (antes del commit de black, isort, prettier y de migración)

* Juntar en 1 solo commit de migración lo que actualmente está en: Migration to 14.0 + dependencies + Migration to 14.0

Me he liado con los commits, disculpa.
Subo ahora un nuevo PR y te etiqueto, gracias.

@Vicenteagf Vicenteagf closed this Aug 17, 2022
@Vicenteagf Vicenteagf deleted the 14.0-mig-delivery_seur branch August 17, 2022 15:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.