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

Syncing from upstream OCA/purchase-workflow (18.0) #1692

Merged
merged 29 commits into from
Mar 1, 2025
Merged

Conversation

bt-admin
Copy link

@bt-admin bt-admin commented Mar 1, 2025

bt_gitbot

JordiBForgeFlow and others added 29 commits February 27, 2025 10:53
The purchase BI analysis is not taking into account the
"Force invoiced" check for showing their quantity to be billed, so it
gives contradictory information from the search filter and the pivot.

This commit adapts the SQL report for taking this into account.
Set force_invoiced field invisible on RFQs (state not in purchase or done)
force_invoiced is now tracked in chatter
- Switch to setUpClass for avoiding repeat the same setup for each test.
- Include context keys for avoiding mail operations overhead.
Currently translated at 100.0% (4 of 4 strings)

Translation: purchase-workflow-17.0/purchase-workflow-17.0-purchase_force_invoiced
Translate-URL: https://translation.odoo-community.org/projects/purchase-workflow-17-0/purchase-workflow-17-0-purchase_force_invoiced/it/
* Reset qty_to_invoice on lines of force invoiced orders
* Adapt SQL report override to the use of odoo.tools.sql.SQL

Co-authored-by: Stefan Rijnhart <[email protected]>
Signed-off-by pedrobaeza
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate.

Translation: purchase-workflow-18.0/purchase-workflow-18.0-purchase_force_invoiced
Translate-URL: https://translation.odoo-community.org/projects/purchase-workflow-18-0/purchase-workflow-18-0-purchase_force_invoiced/
@bt-admin bt-admin added the 18.0 label Mar 1, 2025
@bt-admin bt-admin merged commit dd14698 into brain-tec:18.0 Mar 1, 2025
4 checks passed
Copy link

codecov bot commented Mar 1, 2025

Codecov Report

Attention: Patch coverage is 95.78947% with 4 lines in your changes missing coverage. Please review.

Project coverage is 95.75%. Comparing base (e402810) to head (81e184a).
Report is 53 commits behind head on 18.0.

Files with missing lines Patch % Lines
purchase_force_invoiced/model/purchase_order.py 80.00% 1 Missing and 1 partial ⚠️
purchase_force_invoiced/reports/purchase_report.py 85.71% 1 Missing and 1 partial ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             18.0    #1692      +/-   ##
==========================================
+ Coverage   95.66%   95.75%   +0.09%     
==========================================
  Files         133      141       +8     
  Lines        3298     3393      +95     
  Branches      297      300       +3     
==========================================
+ Hits         3155     3249      +94     
- Misses         69       70       +1     
  Partials       74       74              

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.