-
-
Notifications
You must be signed in to change notification settings - Fork 621
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][FIX] partner_statement: context_today() instead of today() #1280
Conversation
Hi @MiquelRForgeFlow, |
Please note, the tests are failing on another module, seemingly unrelated:
|
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
@AaronHForgeFlow Thank you for the title change. Sorry I will try to keep this notation in mind in the future. |
Thank you. Can you also include the module name in the commit message? |
The date shown on the report should be based on the client's timezone not UTC. (can be off by a day)
Done. |
Thank you @dannyadair /ocabot merge patch |
On my way to merge this fine PR! |
Congratulations, your PR was merged at 225ce9a. Thanks a lot for contributing to OCA. ❤️ |
@dannyadair could you do forward ports? |
Forward-ported to 17.0 in #1282 |
The date shown on the report should be based on the client's timezone not UTC. (can be off by a day)