-
-
Notifications
You must be signed in to change notification settings - Fork 397
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: ux improvements #763
Conversation
Hi @etobella, @alexis-via, |
9155020
to
c82cfa2
Compare
Please rebase. |
c82cfa2
to
e198c74
Compare
Done |
@@ -181,6 +181,11 @@ | |||
string="Transaction Type" | |||
context="{'group_by': 'transaction_type'}" | |||
/> | |||
<filter |
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.
<filter | |
<filter |
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.
Well this is indeed aligned with the above tags formatting... I didn't want to reformat so much to avoid the noise
OK, the whole indentation there is mangled, but let's continue: /ocabot merge minor |
On my way to merge this fine PR! |
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.
Also already approved by others in the previous PR.
Congratulations, your PR was merged at 6070196. Thanks a lot for contributing to OCA. ❤️ |
Superseds: [16.0][IMP] account_reconcile_oca: Open reconcile view when press the name on journal name #730
Minor improvement on account_statement_base to add date grouping.
Depends on fix from [16.0][IMP] account_reconcile_oca: Add auto-reconcile compatibility (Example: Rule to match invoices/bills) #762
fyi @etobella
please review @pedrobaeza @victoralmau
cc @Tecnativa TT52123