-
-
Notifications
You must be signed in to change notification settings - Fork 387
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][IMP] account_reconcile_oca: reconcile lines from statement #757
[16.0][IMP] account_reconcile_oca: reconcile lines from statement #757
Conversation
Why 2 commits? |
7f325cb
to
9f49afd
Compare
Yeah sorry. It's fixed now |
UX: override the statement lines button action so we can reconcile directly from the statement itself. TT51834
9f49afd
to
a3c7914
Compare
Hi @etobella, |
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 patch
On my way to merge this fine PR! |
Congratulations, your PR was merged at 204e473. Thanks a lot for contributing to OCA. ❤️ |
Please fw-port it to 17 |
UX: override the statement lines button action so we can reconcile
directly from the statement itself.
cc @Tecnativa TT51834
please review @pedrobaeza @victoralmau