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
[ADD] account_reconcile_reconciliation_date #299
[ADD] account_reconcile_reconciliation_date #299
Conversation
@osi-scampbell LGTM but it's missing documentation and unit tests |
[IMP] New Line EOF [IMP] Flake8 [ADD] Documentation and Test Cases [IMP] Flake8
771e9c9
to
58a4735
Compare
/ocabot merge |
This PR looks fantastic, let's merge it! |
@osi-scampbell What about partial reconciliations? If you create two payments for the same invoice? As I understand the field in the invoice would show only the date of the last payment. |
@max3903 your merge command was aborted due to failed check(s), which you can inspect on this commit of 12.0-ocabot-merge-pr-299-by-max3903-bump-no. 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. |
Hello @JordiBForgeFlow That's the requirement. @patrickrwilson may confirm. |
/ocabot merge |
What a great day to merge this nice PR. Let's do it! |
This PR has the |
Congratulations, your PR was merged at 90a402d. Thanks a lot for contributing to OCA. ❤️ |
This PR adds account_reconcile_reconciliation_date to the OCA/account-reconcile repo. See issue #294