search

OCA / account-financial-tools

Odoo Accountant Financial Tools and Utils
GNU Affero General Public License v3.0
298 stars 739 forks source link

[14.0][BCK] account_asset_management: KeyError #1871

Closed bosd closed 2 months ago

bosd commented 2 months ago

Backport of #1723

OCA-git-bot commented 2 months ago

On my way to merge this fine PR! Prepared branch 14.0-ocabot-merge-pr-1871-by-pedrobaeza-bump-patch, awaiting test results.

bosd commented 2 months ago

The bot is taking a while. I noticed that GH actions on OCA repo's is very slow as well. (as well as with the runboats) @sbidoul Is something going on with the server?

pedrobaeza commented 2 months ago

/ocabot merge patch

OCA-git-bot commented 2 months ago

On my way to merge this fine PR! Prepared branch 14.0-ocabot-merge-pr-1871-by-pedrobaeza-bump-patch, awaiting test results.

pedrobaeza commented 2 months ago

I think the Codecov situation is not yet resolved, so this won't get merged yet. More details on Discord infrastructure group.

OCA-git-bot commented 2 months ago

@pedrobaeza your merge command was aborted due to failed check(s), which you can inspect on this commit of 14.0-ocabot-merge-pr-1871-by-pedrobaeza-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.

pedrobaeza commented 2 months ago

/ocabot merge patch

OCA-git-bot commented 2 months ago

This PR looks fantastic, let's merge it! Prepared branch 14.0-ocabot-merge-pr-1871-by-pedrobaeza-bump-patch, awaiting test results.

OCA-git-bot commented 2 months ago

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