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_multi_destination: Migration to version 16.0 #766

Merged
merged 23 commits into from
Apr 1, 2024

Commits on Jan 25, 2024

  1. [ADD] delivery_multi_destination

    ==================================================
    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.
    pedrobaeza authored and carolinafernandez-tecnativa committed Jan 25, 2024
    Configuration menu
    Copy the full SHA
    5527442 View commit details
    Browse the repository at this point in the history
  2. [10.0][ADD] delivery_multi_destination

    OCA Transbot updated translations from Transifex
    luismontalba authored and carolinafernandez-tecnativa committed Jan 25, 2024
    Configuration menu
    Copy the full SHA
    d4435dc View commit details
    Browse the repository at this point in the history
  3. [FIX+IMP] delivery_multi_destination: Don't show children + demo (OCA…

    …#148)
    
    * Don't show children carriers on many2one selections
    * Don't search by default children carriers
    * Add demo data
    
    OCA Transbot updated translations from Transifex
    pedrobaeza authored and carolinafernandez-tecnativa committed Jan 25, 2024
    Configuration menu
    Copy the full SHA
    bf114e4 View commit details
    Browse the repository at this point in the history
  4. [MIG] delivery_multi_destination: Migration to 11.0

    * Standard procedure
    * README by fragments
    * Code changed to follow v11 logic
    * Tests adapted and expanded
    pedrobaeza authored and carolinafernandez-tecnativa committed Jan 25, 2024
    Configuration menu
    Copy the full SHA
    c1fbe16 View commit details
    Browse the repository at this point in the history
  5. Configuration menu
    Copy the full SHA
    511b5ba View commit details
    Browse the repository at this point in the history
  6. [FIX] delivery_multi_destination: Fix singleton error

    Incorrect use of variables
    pedrobaeza authored and carolinafernandez-tecnativa committed Jan 25, 2024
    Configuration menu
    Copy the full SHA
    18f6ed1 View commit details
    Browse the repository at this point in the history
  7. [IMP] delivery_multi_destination: Improve view for hiding undesired e…

    …lements
    
    On a multi-destination carrier, that fields shouldn't be shown
    pedrobaeza authored and carolinafernandez-tecnativa committed Jan 25, 2024
    Configuration menu
    Copy the full SHA
    f89f07a View commit details
    Browse the repository at this point in the history
  8. [FIX] delivery_multi_destination: available carriers must return the …

    …main one
    
    When testing available carriers, we must return the main one if one of the
    children matches, not returning the children itself.
    pedrobaeza authored and carolinafernandez-tecnativa committed Jan 25, 2024
    Configuration menu
    Copy the full SHA
    d8ec3a4 View commit details
    Browse the repository at this point in the history
  9. [FIX] delivery_multi_destination: Make delivery shipping properly wit…

    …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.
    pedrobaeza authored and carolinafernandez-tecnativa committed Jan 25, 2024
    Configuration menu
    Copy the full SHA
    8ca3725 View commit details
    Browse the repository at this point in the history
  10. Configuration menu
    Copy the full SHA
    c3e2e00 View commit details
    Browse the repository at this point in the history
  11. [MIG] delivery_multi_destination: Migration to v13.0

    [UPD] Update delivery_multi_destination.pot
    
    [UPD] README.rst
    CarlosRoca13 authored and carolinafernandez-tecnativa committed Jan 25, 2024
    Configuration menu
    Copy the full SHA
    c8135ba View commit details
    Browse the repository at this point in the history
  12. Configuration menu
    Copy the full SHA
    378c929 View commit details
    Browse the repository at this point in the history
  13. [MIG] delivery_multi_destination: Migration to 14.0

    [UPD] Update delivery_multi_destination.pot
    
    [UPD] README.rst
    jado95 authored and carolinafernandez-tecnativa committed Jan 25, 2024
    Configuration menu
    Copy the full SHA
    7130374 View commit details
    Browse the repository at this point in the history
  14. [FIX] delivery_multi_destination: consider no carrier scenario

    When there is no carrier, destination_type is false and can cause error.
    In the case, `one` destination_type should be used.
    
    delivery_multi_destination 14.0.1.0.1
    hailangvn authored and carolinafernandez-tecnativa committed Jan 25, 2024
    Configuration menu
    Copy the full SHA
    f2329b1 View commit details
    Browse the repository at this point in the history
  15. Configuration menu
    Copy the full SHA
    2f56f04 View commit details
    Browse the repository at this point in the history
  16. Configuration menu
    Copy the full SHA
    df3737f View commit details
    Browse the repository at this point in the history
  17. Configuration menu
    Copy the full SHA
    67bc983 View commit details
    Browse the repository at this point in the history
  18. Revert "[FIX] delivery_multi_destination: consider no carrier scenario"

    This reverts commit 0e28544.
    
    This ismixing in the same commit other things for a non related problem that has
    been fixed in a better way in the previous version, so we are reverting it and
    applying the rest of the patches.
    pedrobaeza authored and carolinafernandez-tecnativa committed Jan 25, 2024
    Configuration menu
    Copy the full SHA
    4e58f46 View commit details
    Browse the repository at this point in the history
  19. Configuration menu
    Copy the full SHA
    f004d90 View commit details
    Browse the repository at this point in the history
  20. [FIX] delivery_multi_destination: Get properly subcarrier price on ba…

    …sed on rules
    
    If the destination carrier line is based on rules, the price is not
    correctly fetched, as it's hardcoded to call `_get_price_available`
    using picking's carrier, no matter the recordset from which you call it
    (the self argument).
    
    Thus, the only solution to get the proper value is to temporarily
    replace the carrier on the picking on the calls chain, to restore it
    before returning.
    
    TT42862
    pedrobaeza authored and carolinafernandez-tecnativa committed Jan 25, 2024
    Configuration menu
    Copy the full SHA
    1e42e12 View commit details
    Browse the repository at this point in the history
  21. [FIX] delivery_multi_destination: Filter subdestination carriers from…

    … other companies
    
    In the context where `carrier.child_ids` is being examined, all
    existing subdestinations, no matter the company they have, are shown as
    being in a sudo environment, so we need to filter them out those from
    other companies.
    
    TT43596
    pedrobaeza authored and carolinafernandez-tecnativa committed Jan 25, 2024
    Configuration menu
    Copy the full SHA
    b2dcabb View commit details
    Browse the repository at this point in the history
  22. Configuration menu
    Copy the full SHA
    ab04037 View commit details
    Browse the repository at this point in the history

Commits on Feb 8, 2024

  1. Configuration menu
    Copy the full SHA
    4539c4c View commit details
    Browse the repository at this point in the history