Skip to content

Conversation

@florian-dacosta
Copy link
Contributor

I need to manage kit (phantom bom) sometimes in my RMA.
in rma_sale_mrp, there is already an option at company level allowing to add a rma line with the kit of multiple rma lines with the component, when creating the rma line from a sale order line.

But the to receive/deliver quantity are not managed for this kind of product.

This PR aims to solve it, at least for basic cases.

@florian-dacosta florian-dacosta force-pushed the 16-rma-phantom-bom-qty branch 2 times, most recently from fe9ff5b to 0708128 Compare May 5, 2025 10:12
@codecov-commenter
Copy link

codecov-commenter commented May 5, 2025

Codecov Report

Attention: Patch coverage is 95.83333% with 1 line in your changes missing coverage. Please review.

Project coverage is 82.14%. Comparing base (bb16136) to head (f8f7f75).
Report is 49 commits behind head on 16.0.

Files with missing lines Patch % Lines
rma/models/rma_order_line.py 92.85% 0 Missing and 1 partial ⚠️
Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##             16.0     #618      +/-   ##
==========================================
+ Coverage   81.36%   82.14%   +0.78%     
==========================================
  Files         159      162       +3     
  Lines        5425     5875     +450     
  Branches      874      742     -132     
==========================================
+ Hits         4414     4826     +412     
- Misses        769      807      +38     
  Partials      242      242              
Files with missing lines Coverage Δ
rma_sale_mrp/models/rma_order_line.py 70.37% <100.00%> (+17.42%) ⬆️
rma/models/rma_order_line.py 83.66% <92.85%> (+1.46%) ⬆️

... and 9 files with indirect coverage changes


Continue to review full report in Codecov by Sentry.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 2d6b612...f8f7f75. Read the comment docs.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@florian-dacosta florian-dacosta force-pushed the 16-rma-phantom-bom-qty branch from e624d01 to f8f7f75 Compare June 24, 2025 10:14
@github-actions
Copy link

github-actions bot commented Jan 4, 2026

There hasn't been any activity on this pull request in the past 4 months, so it has been marked as stale and it will be closed automatically if no further activity occurs in the next 30 days.
If you want this PR to never become stale, please ask a PSC member to apply the "no stale" label.

@github-actions github-actions bot added the stale label Jan 4, 2026
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.

2 participants