-
-
Notifications
You must be signed in to change notification settings - Fork 366
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
[18.0] [MIG] delivery_postlogistics: Migration to 18.0 #974
base: 18.0
Are you sure you want to change the base?
[18.0] [MIG] delivery_postlogistics: Migration to 18.0 #974
Conversation
Superseeds #940 |
98eb1c9
to
4d44284
Compare
hello @StephaneMangin
|
/ocabot migration delivery_postlogistics |
delivery_postlogistics/tests/fixtures/cassettes/test_token_error.yaml
Outdated
Show resolved
Hide resolved
delivery_postlogistics/tests/fixtures/cassettes/test_store_label.yaml
Outdated
Show resolved
Hide resolved
delivery_postlogistics/tests/fixtures/cassettes/test_store_label.yaml
Outdated
Show resolved
Hide resolved
delivery_postlogistics/tests/fixtures/cassettes/test_missing_language.yaml
Outdated
Show resolved
Hide resolved
delivery_postlogistics/tests/fixtures/cassettes/test_missing_language.yaml
Outdated
Show resolved
Hide resolved
Yes it is transitional, because these values are added in a dependency. Updating... |
3fe8f87
to
78c780f
Compare
0f171db
to
894894d
Compare
e9dc13c
to
0b24d58
Compare
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate. Translation: delivery-carrier-16.0/delivery-carrier-16.0-delivery_postlogistics Translate-URL: https://translation.odoo-community.org/projects/delivery-carrier-16-0/delivery-carrier-16-0-delivery_postlogistics/
Postlogistics limit for fields such as name, street and city, is 35 chars for the recipient fields but is limited to 25 characters.
Sometimes due to API error it may happen we don't receive anything back when requesting token. In this case we want to show meaningful message.
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate. Translation: delivery-carrier-16.0/delivery-carrier-16.0-delivery_postlogistics Translate-URL: https://translation.odoo-community.org/projects/delivery-carrier-16-0/delivery-carrier-16-0-delivery_postlogistics/
If a stock.picking is created and in the same transaction we generate a label, when the webservice returns an error, the module rollbacks the transaction, so 'self' is empty and will fail with: Record does not exist or has been deleted. (Record: stock.picking(8211067,), User: 6) When trying to write the tracking number, and will therefore not raise the details of the error.
The tests from d5c3f73 failed due to partner city being required
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate. Translation: delivery-carrier-16.0/delivery-carrier-16.0-delivery_postlogistics Translate-URL: https://translation.odoo-community.org/projects/delivery-carrier-16-0/delivery-carrier-16-0-delivery_postlogistics/
22b6efe
to
cc1636f
Compare
cc1636f
to
683d7fa
Compare
res["postlogistics_default_package_type_id"] = self.env.ref( | ||
"delivery_postlogistics.postlogistics_default_package_type", | ||
raise_if_not_found=False, | ||
).id |
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.
The raise_if_not_found will raise a Nonetype if the xmlid is not found
Includes:
postlogistics_send_shipping
requirements (Using ECO default shipping pricing)Depends on:
To avoid Odoo external URL calls restriction, decorate your test class with
@common.tagged("-standard")
(removing the standard tag which is by default). This way, Odoo doesn't patch therequest.Session.send
method to throw an exception in an external URL call case.