Skip to content

[18][UPD] deltatech_sale_payment: store=True, statusuri complete, mig…#2469

Merged
dhongu merged 1 commit into
18.0from
18.0-deltatech_sale_payment
May 22, 2026
Merged

[18][UPD] deltatech_sale_payment: store=True, statusuri complete, mig…#2469
dhongu merged 1 commit into
18.0from
18.0-deltatech_sale_payment

Conversation

@dhongu
Copy link
Copy Markdown
Owner

@dhongu dhongu commented May 22, 2026

…rare SQL

  • store=True pe payment_amount, payment_status, provider_id cu @api.depends complet
  • Status pending adăugat ca stare distinctă
  • Facturi in_payment excluse din payment_amount (evită dubla numărare)
  • payment_amount capped la 0 (evită valori negative din refund-uri fără plată)
  • Decorații complete pentru toate statusurile în formular
  • Filtre complete în lista de comenzi (6 statusuri)
  • action_payment_link: calcul corect pe amount_residual din facturi postate
  • Migrare SQL 18.0.1.2.0: populează payment_amount, payment_status, provider_id pe comenzile existente via sale_order_line_invoice_rel (fix join cartezian)
  • Teste actualizate: product existent din DB, teste noi pentru pending/cancelled/authorized

…rare SQL

- store=True pe payment_amount, payment_status, provider_id cu @api.depends complet
- Status pending adăugat ca stare distinctă
- Facturi in_payment excluse din payment_amount (evită dubla numărare)
- payment_amount capped la 0 (evită valori negative din refund-uri fără plată)
- Decorații complete pentru toate statusurile în formular
- Filtre complete în lista de comenzi (6 statusuri)
- action_payment_link: calcul corect pe amount_residual din facturi postate
- Migrare SQL 18.0.1.2.0: populează payment_amount, payment_status, provider_id
  pe comenzile existente via sale_order_line_invoice_rel (fix join cartezian)
- Teste actualizate: product existent din DB, teste noi pentru pending/cancelled/authorized

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@codecov
Copy link
Copy Markdown

codecov Bot commented May 22, 2026

Codecov Report

❌ Patch coverage is 82.53968% with 11 lines in your changes missing coverage. Please review.
✅ Project coverage is 72.69%. Comparing base (6aaa07e) to head (65ec9d8).

Files with missing lines Patch % Lines
deltatech_sale_payment/models/sale.py 79.54% 6 Missing and 3 partials ⚠️
deltatech_sale_payment/tests/test_sale.py 89.47% 1 Missing and 1 partial ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             18.0    #2469      +/-   ##
==========================================
+ Coverage   72.63%   72.69%   +0.05%     
==========================================
  Files        1229     1229              
  Lines       19874    19876       +2     
  Branches     2803     2801       -2     
==========================================
+ Hits        14436    14448      +12     
+ Misses       4633     4622      -11     
- Partials      805      806       +1     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@dhongu dhongu merged commit 7bacf79 into 18.0 May 22, 2026
5 checks passed
@dhongu dhongu deleted the 18.0-deltatech_sale_payment branch May 22, 2026 02:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant