Skip to content

[11.0] [FIX] _compute_tax_group_id: Filter taxes by company#50

Open
hugosantosred wants to merge 1 commit intoOCA:11.0from
factorlibre:11.0-fix-compute_tax_group_id
Open

[11.0] [FIX] _compute_tax_group_id: Filter taxes by company#50
hugosantosred wants to merge 1 commit intoOCA:11.0from
factorlibre:11.0-fix-compute_tax_group_id

Conversation

@hugosantosred
Copy link
Member

Filter product taxes by the selected company in user to avoid singleton
error if product has taxes for more than one company and the method is
called with admin user

Filter product taxes by the selected company in user to avoid singleton
error if product has taxes for more than one company and the method is
called with admin user
@coveralls
Copy link

Coverage Status

Coverage decreased (-0.2%) to 58.434% when pulling ffc8b94 on factorlibre:11.0-fix-compute_tax_group_id into 5ba71ba on OCA:11.0.

@gurneyalex gurneyalex added this to the 11.0 milestone Jan 6, 2022
@gurneyalex
Copy link
Member

/ocabot merge patch

@OCA-git-bot
Copy link
Contributor

This PR looks fantastic, let's merge it!
Prepared branch 11.0-ocabot-merge-pr-50-by-gurneyalex-bump-patch, awaiting test results.

OCA-git-bot added a commit that referenced this pull request Jan 6, 2022
Signed-off-by gurneyalex
@OCA-git-bot
Copy link
Contributor

@gurneyalex your merge command was aborted due to failed check(s), which you can inspect on this commit of 11.0-ocabot-merge-pr-50-by-gurneyalex-bump-patch.

After fixing the problem, you can re-issue a merge command. Please refrain from merging manually as it will most probably make the target branch red.

@OCA-git-bot
Copy link
Contributor

This PR has the approved label and has been created more than 5 days ago. It should therefore be ready to merge by a maintainer (or a PSC member if the concerned addon has no declared maintainer). 🤖

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.

6 participants