Skip to content

[MIG] sale_margin_delivery_cost: Migration to 19.0 - #280

Open
dreispt wants to merge 7 commits into
OCA:19.0from
dreispt:19.0-mig-sale_margin_delivery_cost
Open

[MIG] sale_margin_delivery_cost: Migration to 19.0#280
dreispt wants to merge 7 commits into
OCA:19.0from
dreispt:19.0-mig-sale_margin_delivery_cost

Conversation

@dreispt

@dreispt dreispt commented Aug 18, 2026

Copy link
Copy Markdown
Member

Summary

This migrates sale_margin_delivery_cost from 16.0 to 19.0.

Migration changes:

  • Update the sale.order form view to use <list> and the Odoo 17+ readonly expression syntax (replacing <tree> and attrs).
  • Replace the deprecated ('type', '!=', 'product') product domain with the current product type selection values ('type', 'in', ('consu', 'service')).
  • Convert README fragments to markdown and regenerate README/index for 19.0.

Assisted-by: Devin:SWE-1.7

@OCA-git-bot OCA-git-bot added series:19.0 mod:sale_margin_delivery_cost Module sale_margin_delivery_cost labels Aug 18, 2026
@devin-ai-integration
devin-ai-integration Bot force-pushed the 19.0-mig-sale_margin_delivery_cost branch from a0d993c to b08ee5d Compare August 18, 2026 09:37
Port sale_margin_delivery_cost from 16.0 to 19.0.

Migration changes:
- Update the Sale Order view to use <list> and the Odoo 17+ readonly expression syntax.
- Replace the deprecated product type exclusion domain with the current product type selection values.

Assisted-by: Devin:SWE-1.7
Add tests covering:
- margin computation with other costs
- set_delivery_line for "other" charging policy
- set_delivery_line for "sale" charging policy

Assisted-by: Devin:SWE-1.7
@devin-ai-integration
devin-ai-integration Bot force-pushed the 19.0-mig-sale_margin_delivery_cost branch from b08ee5d to 7c39ebc Compare August 18, 2026 10:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

mod:sale_margin_delivery_cost Module sale_margin_delivery_cost series:19.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants