Skip to content

[15.0][MIG] purchase_triple_discount#1437

Merged
OCA-git-bot merged 33 commits intoOCA:15.0from
ramiadavid:15.0-mig-purchase_triple_discount
Oct 11, 2022
Merged

[15.0][MIG] purchase_triple_discount#1437
OCA-git-bot merged 33 commits intoOCA:15.0from
ramiadavid:15.0-mig-purchase_triple_discount

Conversation

@ramiadavid
Copy link
Copy Markdown
Contributor

@ramiadavid ramiadavid commented Apr 26, 2022

@ramiadavid ramiadavid force-pushed the 15.0-mig-purchase_triple_discount branch from 4a5bd13 to 8689c26 Compare May 22, 2022 22:56
@ramiadavid ramiadavid mentioned this pull request May 22, 2022
78 tasks
@ramiadavid ramiadavid changed the title [MIG] purchase_triple_discount: Migration to 15.0 [15.0][MIG] purchase_triple_discount: May 23, 2022
@ramiadavid ramiadavid changed the title [15.0][MIG] purchase_triple_discount: [15.0][MIG] purchase_triple_discount May 23, 2022
@ramiadavid ramiadavid force-pushed the 15.0-mig-purchase_triple_discount branch 2 times, most recently from 98acbb5 to 31a0c12 Compare July 5, 2022 17:53
@ramiadavid ramiadavid force-pushed the 15.0-mig-purchase_triple_discount branch from 31a0c12 to ba3bdb0 Compare September 6, 2022 21:18
@ramiadavid
Copy link
Copy Markdown
Contributor Author

Can someone review this PR?

@ramiadavid ramiadavid force-pushed the 15.0-mig-purchase_triple_discount branch from ba3bdb0 to 52e6bf8 Compare October 2, 2022 15:40
@oca-clabot
Copy link
Copy Markdown

Hey @ramiadavid, thank you for your Pull Request.

It looks like some users haven't signed our Contributor License Agreement, yet.
You can read and sign our full Contributor License Agreement here: https://odoo-community.org/page/cla
Here is a list of the users:

Appreciation of efforts,
OCA CLAbot

@ao-landoo
Copy link
Copy Markdown
Contributor

@ramiadavid Can you add this commit please?
#1587

chienandalu and others added 18 commits October 4, 2022 13:39
- remove dependency to obsolete module product_supplierinfo_discount
- rename procurement.rule into stock.rule
- Use new hook introduced in purchase_discount V12 to add discount2 and discount3
- Introduce discount2 and discount3 in purchase.report model
* Lighter algorithm when synchronizing PO line info to product.supplierinfo.
* Use new create multi method for performance.
* Better SQL expresion in purchase report for decreasing the number of operations.
* Add test for purchase report.
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate.

Translation: purchase-workflow-12.0/purchase-workflow-12.0-purchase_triple_discount
Translate-URL: https://translation.odoo-community.org/projects/purchase-workflow-12-0/purchase-workflow-12-0-purchase_triple_discount/
Currently translated at 100.0% (15 of 15 strings)

Translation: purchase-workflow-12.0/purchase-workflow-12.0-purchase_triple_discount
Translate-URL: https://translation.odoo-community.org/projects/purchase-workflow-12-0/purchase-workflow-12-0-purchase_triple_discount/zh_CN/
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate.

Translation: purchase-workflow-12.0/purchase-workflow-12.0-purchase_triple_discount
Translate-URL: https://translation.odoo-community.org/projects/purchase-workflow-12-0/purchase-workflow-12-0-purchase_triple_discount/
Currently translated at 100.0% (15 of 15 strings)

Translation: purchase-workflow-12.0/purchase-workflow-12.0-purchase_triple_discount
Translate-URL: https://translation.odoo-community.org/projects/purchase-workflow-12-0/purchase-workflow-12-0-purchase_triple_discount/it/
@ramiadavid ramiadavid force-pushed the 15.0-mig-purchase_triple_discount branch from 52e6bf8 to 6b80937 Compare October 4, 2022 11:48
@ramiadavid ramiadavid force-pushed the 15.0-mig-purchase_triple_discount branch from 6b80937 to c6ef161 Compare October 4, 2022 12:27
@ramiadavid
Copy link
Copy Markdown
Contributor Author

@ao-landoo Done, but the report in v15 has changed, I had to modify it

Copy link
Copy Markdown
Contributor

@ao-landoo ao-landoo left a comment

Choose a reason for hiding this comment

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

Technical approval

@OCA-git-bot
Copy link
Copy Markdown
Contributor

This PR has the approved label and has been created more than 5 days ago. It should therefore be ready to merge by a maintainer (or a PSC member if the concerned addon has no declared maintainer). 🤖

Copy link
Copy Markdown
Contributor

@manuelregidor manuelregidor left a comment

Choose a reason for hiding this comment

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

LGTM

@HaraldPanten
Copy link
Copy Markdown
Contributor

Hi @rousseldenis @gurneyalex @JordiBForgeFlow could you merge this one?

@bi-landoo
Copy link
Copy Markdown

Hi @rousseldenis @gurneyalex @JordiBForgeFlow @pedrobaeza could you please merge it?

Thanks!

@pedrobaeza
Copy link
Copy Markdown
Member

Hello from OCA days!

/ocabot migration purchase_triple_discount
/ocabot merge nobump

@OCA-git-bot OCA-git-bot added this to the 15.0 milestone Oct 11, 2022
@OCA-git-bot
Copy link
Copy Markdown
Contributor

Hey, thanks for contributing! Proceeding to merge this for you.
Prepared branch 15.0-ocabot-merge-pr-1437-by-pedrobaeza-bump-nobump, awaiting test results.

@OCA-git-bot OCA-git-bot merged commit 243b69b into OCA:15.0 Oct 11, 2022
@OCA-git-bot
Copy link
Copy Markdown
Contributor

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

@HaraldPanten
Copy link
Copy Markdown
Contributor

Hello from OCA days!

/ocabot migration purchase_triple_discount /ocabot merge nobump

Thank you Pedro! 🎉

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.