-
-
Notifications
You must be signed in to change notification settings - Fork 527
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_payment_order_return #1359
[16.0][MIG] account_payment_order_return #1359
Conversation
:100644 100644 d46b1318 a23b471a M account_payment_order_return/README.rst :100644 100644 340d1c77 1235bcb1 M account_payment_order_return/__manifest__.py :100644 100644 a341cc9d be7d7bbb M account_payment_order_return/i18n/account_payment_order_return.pot :100644 100644 60be6ab8 e6b32b37 M account_payment_order_return/readme/CONTRIBUTORS.rst :100644 100644 a8591d03 826c26df M account_payment_order_return/readme/USAGE.rst :100644 100644 450f2d9e a5f5d975 M account_payment_order_return/static/description/index.html
fix domain in payment_order_return in order to avoid problems with moves without associated invoice as discused at OCA#718
Don't post the invoice if it is already posted. See odoo/odoo@80c2818
When the test hour is around midnight, and demo data using CET/CEST timezones, current code fails due to some asserts not having this into consideration. This fix avoids such problem, and irons the execution no matter the hour.
- Include context keys for avoiding mail operations overhead.
8a6c1e6
to
6a5fc8f
Compare
/ocabot migration account_payment_order_return |
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
Thanks for your contribution @almumu
About the README, it is automatically constructed, you should add yourself to account_payment_order_return/readme/CONTRIBUTORS.rst
or next time the README is generated your name will be lost.
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.
/ocabot merge nobump
What a great day to merge this nice PR. Let's do it! |
@pedrobaeza your merge command was aborted due to failed check(s), which you can inspect on this commit of 16.0-ocabot-merge-pr-1359-by-pedrobaeza-bump-nobump. 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. |
/ocabot merge nobump |
What a great day to merge this nice PR. Let's do it! |
Congratulations, your PR was merged at 6b3dee6. Thanks a lot for contributing to OCA. ❤️ |
Migration to 16.0