operating-unit icon indicating copy to clipboard operation
operating-unit copied to clipboard

[15.0][IMP]mis_builder_operating_unit Filter account entries not having operating unit

Open AaronHForgeFlow opened this issue 1 year ago • 8 comments

FW #651

AaronHForgeFlow avatar Mar 06 '24 10:03 AaronHForgeFlow

There is some issue in the account_operating_unit module, tests are failing

AaronHForgeFlow avatar Mar 06 '24 11:03 AaronHForgeFlow

There is some issue in the account_operating_unit module, tests are failing

The issue is raised as

raise ValidationError(_("The selected payment method is not available for this payment, please select the payment method again."))

see here https://github.com/OCA/operating-unit/actions/runs/8170230486/job/22335955970?pr=653#step:8:733

robyf70 avatar Mar 06 '24 14:03 robyf70

I will check when having some time.

AaronHForgeFlow avatar Mar 06 '24 15:03 AaronHForgeFlow

I will check when having some time.

I've committed the fix see #654

robyf70 avatar Mar 06 '24 16:03 robyf70

Thank you @robyf70 this is green now :)

AaronHForgeFlow avatar Mar 07 '24 09:03 AaronHForgeFlow

Thank you @robyf70 this is green now :)

Any chance to get it merged?

robyf70 avatar Apr 09 '24 13:04 robyf70

@AaronHForgeFlow Can apply the changes as per #651 so we have all uniform?

robyf70 avatar Apr 15 '24 18:04 robyf70

@robyf70 sure! in any case this should not me merged until the other is merged

AaronHForgeFlow avatar Apr 16 '24 07:04 AaronHForgeFlow

keeping branches even /ocabot merge minor

AaronHForgeFlow avatar Aug 26 '24 09:08 AaronHForgeFlow

Hey, thanks for contributing! Proceeding to merge this for you. Prepared branch 15.0-ocabot-merge-pr-653-by-AaronHForgeFlow-bump-minor, awaiting test results.

OCA-git-bot avatar Aug 26 '24 09:08 OCA-git-bot

Congratulations, your PR was merged at 1af1120365e51e5917778fc66bac41909744dc2a. Thanks a lot for contributing to OCA. ❤️

OCA-git-bot avatar Aug 26 '24 09:08 OCA-git-bot