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

14.0 mig delivery multi destination #373

Merged
merged 22 commits into from
Jul 3, 2021

Conversation

jado95
Copy link
Contributor

@jado95 jado95 commented Jun 4, 2021

No description provided.

pedrobaeza and others added 21 commits June 4, 2021 12:18
==================================================
Multiple destinations for the same delivery method
==================================================

Module `delivery` in version 8 allows to set different price rules depending
on the destination. This is what is called a delivery grid.

In version 9, for simplifying delivery methods, Odoo has plained the structure,
lowering destinations at delivery method level, and removing delivery grid
model.

This is not usable when you have different prices according the destination
of your delivery.

This module restores the same concept, reusing the same model for nesting
several "children" delivery methods, one per possible destination. It has been
designed to reuse all possible extensions to the base delivery, without the
need to create a glue module for having multiple destinations.

This module also handles if you're migrating from version 8 and you had
`delivery` module installed, to keep the delivery grids.

Installation
============

If you installed the module on a version 8 migrated database, some operations
will be done for recovering delivery grids. If so, you need to have
**openupgradelib** library installed.

Configuration
=============

To configure delivery methods with multiple destinations:

* Go to Inventory > Configuration > Delivery > Delivery Methods
* Create or edit an existing record.
* Select "Destination type" = "Multiple destinations".
* Introduce a line for each destination in the new tab "Destinations"
* Lines have priority, so you have to put first the lines with more restricted
  destinations.

Usage
=====

* When using the delivery method in a Sales order, delivery address will be
  used for computing the delivery price according introduced destinations.
…#148)

* Don't show children carriers on many2one selections
* Don't search by default children carriers
* Add demo data
* Standard procedure
* README by fragments
* Code changed to follow v11 logic
* Tests adapted and expanded
Currently translated at 100.0% (16 of 16 strings)

Translation: delivery-carrier-12.0/delivery-carrier-12.0-delivery_multi_destination
Translate-URL: https://translation.odoo-community.org/projects/delivery-carrier-12-0/delivery-carrier-12-0-delivery_multi_destination/zh_CN/
Currently translated at 100.0% (16 of 16 strings)

Translation: delivery-carrier-12.0/delivery-carrier-12.0-delivery_multi_destination
Translate-URL: https://translation.odoo-community.org/projects/delivery-carrier-12-0/delivery-carrier-12-0-delivery_multi_destination/pt_BR/
Currently translated at 56.2% (9 of 16 strings)

Translation: delivery-carrier-12.0/delivery-carrier-12.0-delivery_multi_destination
Translate-URL: https://translation.odoo-community.org/projects/delivery-carrier-12-0/delivery-carrier-12-0-delivery_multi_destination/de/
…lements

On a multi-destination carrier, that fields shouldn't be shown
…main one

When testing available carriers, we must return the main one if one of the
children matches, not returning the children itself.
…h multi

Handle the step of sending the shipping (triggered on picking validation) for
multi destination delivery. This means to manually check for fixing prices (as
the implementation doesn't check this part and always take the main carrier price,
not the subcarrier one.

Test for this use case done, although the rest of the cases are not covered by
tests yet.
Currently translated at 5.9% (1 of 17 strings)

Translation: delivery-carrier-12.0/delivery-carrier-12.0-delivery_multi_destination
Translate-URL: https://translation.odoo-community.org/projects/delivery-carrier-12-0/delivery-carrier-12-0-delivery_multi_destination/sl/
@jado95 jado95 mentioned this pull request Jun 4, 2021
21 tasks
@jado95 jado95 force-pushed the 14.0-mig-delivery_multi_destination branch from 24756df to 0d3847c Compare June 4, 2021 12:23
@jado95 jado95 force-pushed the 14.0-mig-delivery_multi_destination branch from 0d3847c to 1849529 Compare June 4, 2021 13:02
Copy link

@andreampiovesana andreampiovesana left a comment

Choose a reason for hiding this comment

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

go

@dreispt
Copy link
Sponsor Member

dreispt commented Jul 3, 2021

/ocabot merge nobump

@OCA-git-bot
Copy link
Contributor

Hey, thanks for contributing! Proceeding to merge this for you.
Prepared branch 14.0-ocabot-merge-pr-373-by-dreispt-bump-nobump, awaiting test results.

@OCA-git-bot OCA-git-bot merged commit 42daf5b into OCA:14.0 Jul 3, 2021
@OCA-git-bot
Copy link
Contributor

Congratulations, your PR was merged at f4baa03. 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.