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

[17.0][MIG] sale_blanket_order #3255

Merged
merged 48 commits into from
Aug 20, 2024

Conversation

chrisb-c01
Copy link

@chrisb-c01 chrisb-c01 commented Jul 29, 2024

Based on / supersedes #3014

andreparames and others added 30 commits July 26, 2024 15:03
When duplicating a confirmed blanket order, the new copy shouldn't
keep the state nor the sequence number (name).
Steps to reproduce:

* create and confirm a blanket order (BO) with (product A, qty 30) and (product B, qty 20)
* from the BO create a SO with (product A, qty 10) and (product B, qty 10)
* from the BO create a SO with (product A, qty 20) and (product B, qty 0)
* from the BO create another SO with (product B, qty 10)

Current behavior:

It raises the exception "The sale has already been completed.".

Expected behavior:

No exception is raised.
Currently translated at 7.1% (11 of 154 strings)

Translation: sale-workflow-12.0/sale-workflow-12.0-sale_blanket_order
Translate-URL: https://translation.odoo-community.org/projects/sale-workflow-12-0/sale-workflow-12-0-sale_blanket_order/de/
* Allow to select an analitic account in the blanket order. It
  will be passed on to the SO.
* Allow to select analytic tags on blanket order lines. They will
  be passed to the SO lines.
* Source document was not being filled in SO created from a
  blanket order.
mymage and others added 4 commits July 26, 2024 15:17
Currently translated at 100.0% (172 of 172 strings)

Translation: sale-workflow-16.0/sale-workflow-16.0-sale_blanket_order
Translate-URL: https://translation.odoo-community.org/projects/sale-workflow-16-0/sale-workflow-16-0-sale_blanket_order/es/
Currently translated at 100.0% (170 of 170 strings)

Translation: sale-workflow-16.0/sale-workflow-16.0-sale_blanket_order
Translate-URL: https://translation.odoo-community.org/projects/sale-workflow-16-0/sale-workflow-16-0-sale_blanket_order/it/
@OCA-git-bot
Copy link
Contributor

Hi @smaciaosi, @dreispt, @ckolobow,
some modules you are maintaining are being modified, check this out!

@chrisb-c01
Copy link
Author

@OCA/core-maintainers can someone review this PR?

@chrisb-c01
Copy link
Author

@sanderlienaerts can you review?

Copy link

@sanderlienaerts sanderlienaerts left a comment

Choose a reason for hiding this comment

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

The migration does not work properly on Runboat and locally due to lines with: column_invisible="display_type"

sale_blanket_order/views/sale_config_settings.xml Outdated Show resolved Hide resolved
sale_blanket_order/views/sale_blanket_order_views.xml Outdated Show resolved Hide resolved
sale_blanket_order/views/sale_blanket_order_views.xml Outdated Show resolved Hide resolved
sale_blanket_order/views/sale_blanket_order_views.xml Outdated Show resolved Hide resolved
@chrisb-c01 chrisb-c01 force-pushed the 17.0-mig-sale_blanket_order branch from cb61297 to e48f1b6 Compare August 2, 2024 14:21
@chrisb-c01
Copy link
Author

@sanderlienaerts can you please retest?

Copy link

@sanderlienaerts sanderlienaerts left a comment

Choose a reason for hiding this comment

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

LGTM

@sanderlienaerts
Copy link

Hi @SMaciasOSI, tagging you here since the bot was unable to

@rousseldenis
Copy link
Contributor

/ocabot migration sale_blanket_order

@dreispt
Copy link
Member

dreispt commented Aug 20, 2024

/ocabot merge nobump

@OCA-git-bot
Copy link
Contributor

This PR looks fantastic, let's merge it!
Prepared branch 17.0-ocabot-merge-pr-3255-by-dreispt-bump-nobump, awaiting test results.

@OCA-git-bot OCA-git-bot merged commit a6859e2 into OCA:17.0 Aug 20, 2024
5 of 7 checks passed
@OCA-git-bot
Copy link
Contributor

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

@sanderlienaerts sanderlienaerts deleted the 17.0-mig-sale_blanket_order branch September 6, 2024 08:56
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.