Merge lp:~camptocamp/banking-addons/bank-statement-reconcile-7.0-improv-controlling-jge into lp:banking-addons/bank-statement-reconcile-70
Proposed by
Joël Grand-Guillaume @ camptocamp
Status: | Merged |
---|---|
Approved by: | Yannick Vaucher @ Camptocamp |
Approved revision: | 138 |
Merged at revision: | 139 |
Proposed branch: | lp:~camptocamp/banking-addons/bank-statement-reconcile-7.0-improv-controlling-jge |
Merge into: | lp:banking-addons/bank-statement-reconcile-70 |
Diff against target: |
169 lines (+88/-8) 5 files modified
account_easy_reconcile/easy_reconcile.py (+53/-0) account_easy_reconcile/easy_reconcile.xml (+10/-4) account_statement_base_completion/statement.py (+7/-4) account_statement_ext/__openerp__.py (+1/-0) account_statement_ext/account_view.xml (+17/-0) |
To merge this branch: | bzr merge lp:~camptocamp/banking-addons/bank-statement-reconcile-7.0-improv-controlling-jge |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Yannick Vaucher @ Camptocamp | code review, no test | Approve | |
Leonardo Pistone | code review | Approve | |
Review via email: mp+209901@code.launchpad.net |
Description of the change
Hi,
Those improvements mostely concerns various imrpovements to ease the controlling process:
[IMP] Add buttons to open unreconciled and partially reconciled items from a profile to easy the verification and controlling
[IMP] Add the number of lines in completion log to let the user know if some hasn't been auto-completed (e.g. 332/335 line compelted)
[IMP] Add a group by bank statement in journal items search view to ease the reconciliation
To post a comment you must log in.
The code LGTM.
I would prefer to have one commit for each module, but I am not blocking because of that (I find it convenient to always use "bzr commit -i" from the interactive plugin, but maybe it is just because I miss "git add -p").
Thanks!