Skip to content

[16.0][FIX] sale_order_invoice_amount: Fix invoice amount calc when order a…#4075

Merged
OCA-git-bot merged 1 commit intoOCA:16.0from
factorlibre:16.0-fix-sale_order_invoice_amount
Jan 27, 2026
Merged

[16.0][FIX] sale_order_invoice_amount: Fix invoice amount calc when order a…#4075
OCA-git-bot merged 1 commit intoOCA:16.0from
factorlibre:16.0-fix-sale_order_invoice_amount

Conversation

@aritzolea
Copy link
Copy Markdown

…nd invoice have foreign currencies and fix tests

@@ -314,7 +314,7 @@ def test_03_sale_order_invoiced_amount_different_currencies_sale(self):
self.assertAlmostEqual(
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

🤯

@aritzolea aritzolea force-pushed the 16.0-fix-sale_order_invoice_amount branch from 84b2572 to 98be8df Compare December 22, 2025 13:11
…nd invoice have foreign currencies and fix tests
@aritzolea aritzolea force-pushed the 16.0-fix-sale_order_invoice_amount branch from 98be8df to c5c706d Compare December 22, 2025 14:24
@aritzolea aritzolea changed the title [FIX] sale_order_invoice_amount: Fix invoice amount calc when order a… [16.0][FIX] sale_order_invoice_amount: Fix invoice amount calc when order a… Dec 30, 2025
@aritzolea aritzolea marked this pull request as ready for review January 8, 2026 14:46
@aritzolea
Copy link
Copy Markdown
Author

Hi @rousseldenis could you review and merge this if OK? Thanks in advance

Copy link
Copy Markdown
Contributor

@rousseldenis rousseldenis left a comment

Choose a reason for hiding this comment

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

Seems legit

@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). 🤖

@aritzolea
Copy link
Copy Markdown
Author

aritzolea commented Jan 13, 2026

Seems legit

Could you merge @rousseldenis ?

@rousseldenis
Copy link
Copy Markdown
Contributor

/ocabot merge patch

@OCA-git-bot
Copy link
Copy Markdown
Contributor

Hey, thanks for contributing! Proceeding to merge this for you.
Prepared branch 16.0-ocabot-merge-pr-4075-by-rousseldenis-bump-patch, awaiting test results.

@OCA-git-bot OCA-git-bot merged commit d86b09a into OCA:16.0 Jan 27, 2026
15 checks passed
@OCA-git-bot
Copy link
Copy Markdown
Contributor

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

@aritzolea aritzolea deleted the 16.0-fix-sale_order_invoice_amount branch January 28, 2026 07:44
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.

5 participants