-
-
Notifications
You must be signed in to change notification settings - Fork 526
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
[16.0][MIG] account_banking_mandate_contact: Migration to 16.0. #1233
[16.0][MIG] account_banking_mandate_contact: Migration to 16.0. #1233
Conversation
- Switch to setUpClass for avoiding repeat the same setup for each test. - Include context keys for avoiding mail operations overhead.
f89ac13
to
7cbd626
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM 👍🏻
7cbd626
to
4298db0
Compare
@HaraldPanten could u review please? 😄 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Functional review in runboat. LGTM
/ocabot merge nobump |
On my way to merge this fine PR! |
Congratulations, your PR was merged at 3b66e9e. Thanks a lot for contributing to OCA. ❤️ |
No description provided.