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/commission (16.0) #483

Merged
merged 3 commits into from
Aug 23, 2024
Merged

Conversation

bt-admin
Copy link

bt_gitbot

CLaurelB and others added 3 commits August 22, 2024 13:27
…move lines

Fetch the partner by move line record instead of fetching all partners
for all invoice users related to the moves. The previous method could
fetch more than one partner, including those not related to the move,
when multiple lines for different moves are being processed, causing an
error when trying to obtain the values of the `agent` and
`salesman_as_agent` fields.
Signed-off-by pedrobaeza
@bt-admin bt-admin added the 16.0 label Aug 23, 2024
@bt-admin bt-admin merged commit 9524cb4 into brain-tec:16.0 Aug 23, 2024
4 checks passed
Copy link

codecov bot commented Aug 23, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 94.58%. Comparing base (d5dacae) to head (16d720d).
Report is 65 commits behind head on 16.0.

Additional details and impacted files
@@           Coverage Diff           @@
##             16.0     #483   +/-   ##
=======================================
  Coverage   94.58%   94.58%           
=======================================
  Files          80       80           
  Lines        2288     2288           
  Branches      270      270           
=======================================
  Hits         2164     2164           
  Misses         83       83           
  Partials       41       41           

☔ 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.

3 participants