Skip to content

[16.0][FIX] account_credit_control: avoid AccessError in open_credit_lines()#287

Merged
OCA-git-bot merged 1 commit intoOCA:16.0from
onesteinbv:16_fix_account_credit_control
Jun 5, 2023
Merged

[16.0][FIX] account_credit_control: avoid AccessError in open_credit_lines()#287
OCA-git-bot merged 1 commit intoOCA:16.0from
onesteinbv:16_fix_account_credit_control

Conversation

@astirpe
Copy link
Copy Markdown
Member

@astirpe astirpe commented Jun 5, 2023

This fixes the error described by @grindtildeath in #268 (comment)

astirpe added a commit to onesteinbv/addons-oca that referenced this pull request Jun 5, 2023
@nilshamerlinck
Copy link
Copy Markdown

Hello @astirpe might be worth taking the occasion to fix open_credit_communications() as well?

@astirpe
Copy link
Copy Markdown
Member Author

astirpe commented Jun 5, 2023

@nilshamerlinck you are right. Done!

Copy link
Copy Markdown

@Yadier-Tecnativa Yadier-Tecnativa left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@pedrobaeza
Copy link
Copy Markdown
Member

/ocabot merge patch

@OCA-git-bot
Copy link
Copy Markdown
Contributor

What a great day to merge this nice PR. Let's do it!
Prepared branch 16.0-ocabot-merge-pr-287-by-pedrobaeza-bump-patch, awaiting test results.

@OCA-git-bot OCA-git-bot merged commit 363d470 into OCA:16.0 Jun 5, 2023
@OCA-git-bot
Copy link
Copy Markdown
Contributor

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

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