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

[FIX] 14.0 cash rounding on invoices #53

Open
wants to merge 4 commits into
base: 14.0
Choose a base branch
from

Conversation

PierrickBrun
Copy link
Member

No description provided.

@codecov
Copy link

codecov bot commented Feb 17, 2023

Codecov Report

Merging #53 (f4a913d) into 14.0 (1003693) will increase coverage by 0.21%.
Report is 70 commits behind head on 14.0.
The diff coverage is 84.55%.

@@            Coverage Diff             @@
##             14.0      #53      +/-   ##
==========================================
+ Coverage   88.34%   88.56%   +0.21%     
==========================================
  Files          50       51       +1     
  Lines         884     1014     +130     
  Branches      103      159      +56     
==========================================
+ Hits          781      898     +117     
- Misses         90       99       +9     
- Partials       13       17       +4     
Files Coverage Δ
pos_sale_order/models/pos_payment.py 100.00% <100.00%> (ø)
pos_sale_order/tests/common.py 95.45% <100.00%> (+0.21%) ⬆️
pos_sale_order/tests/test_closing_session.py 100.00% <100.00%> (ø)
pos_sale_order/tests/test_create_order.py 100.00% <100.00%> (ø)
pos_sale_order/wizards/__init__.py 100.00% <100.00%> (ø)
pos_sale_order_debug/tests/test_create_order.py 100.00% <100.00%> (ø)
pos_sale_order_delivery/tests/test_order.py 100.00% <100.00%> (ø)
pos_sale_order_load/tests/test_load_order.py 100.00% <100.00%> (ø)
...ale_order_no_payment/tests/test_closing_session.py 100.00% <100.00%> (ø)
pos_sale_order/models/sale_order.py 84.27% <83.33%> (+0.61%) ⬆️
... and 6 more

... and 3 files with indirect coverage changes

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

if there is cash then bank, no rounding
should be applied
when you pay first by gift card then you pay by cash
cash rounding should be taken in account.
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.

3 participants