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

[16.0][MIG] delivery_price_method: Migration to 16.0 #554

Merged
merged 15 commits into from
Nov 15, 2022

Conversation

chienandalu
Copy link
Member

Standard migration

cc @Tecnativa TT40425

please review @victoralmau @pedrobaeza

rlizana and others added 15 commits November 9, 2022 16:50
- Add default price_method called "Carrier obtained price", for being more explicit
  and avoid user confusions.
- Shipment rate: Make it compatible with upstream code tricking the delivery_type
  field.
- Shipping sending: Call upstream carrier sending routine + call standard one, and
  merge results (excluding the tracking_number).
- Tests:

  * Make them more resilient (fixing the pricelist).
  * Use SavepointCase for executing setup only once.
  * Fine tune some code
- Improve README

[UPD] Update delivery_price_method.pot
[UPD] Update delivery_price_method.pot

[UPD] README.rst
If a user don't have the necessary permissions for writing in the
`delivery.carrier` model (like a low range salesman) he won't be able to
choose a carrier wich uses the rate shipment override.

TT31627

delivery_price_method 13.0.1.0.1
[UPD] README.rst

delivery_price_method 13.0.1.0.2
Currently translated at 100.0% (6 of 6 strings)

Translation: delivery-carrier-13.0/delivery-carrier-13.0-delivery_price_method
Translate-URL: https://translation.odoo-community.org/projects/delivery-carrier-13-0/delivery-carrier-13-0-delivery_price_method/es/
Currently translated at 100.0% (6 of 6 strings)

Translation: delivery-carrier-13.0/delivery-carrier-13.0-delivery_price_method
Translate-URL: https://translation.odoo-community.org/projects/delivery-carrier-13-0/delivery-carrier-13-0-delivery_price_method/es_CL/
TT36947

[UPD] Update delivery_price_method.pot

[UPD] README.rst
TT36947

Co-authored-by: jaredkipe
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.

Code review OK.

@chienandalu
Copy link
Member Author

This is RTM

@pedrobaeza
Copy link
Member

/ocabot migration delivery_price_method

@OCA-git-bot OCA-git-bot added this to the 16.0 milestone Nov 15, 2022
@OCA-git-bot OCA-git-bot mentioned this pull request Nov 15, 2022
32 tasks
@pedrobaeza
Copy link
Member

/ocabot merge nobump

@OCA-git-bot
Copy link
Contributor

What a great day to merge this nice PR. Let's do it!
Prepared branch 16.0-ocabot-merge-pr-554-by-pedrobaeza-bump-nobump, awaiting test results.

@OCA-git-bot OCA-git-bot merged commit f8ae4e4 into OCA:16.0 Nov 15, 2022
@OCA-git-bot
Copy link
Contributor

Congratulations, your PR was merged at 130afe2. Thanks a lot for contributing to OCA. ❤️

This pull request was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants