Banking Addons

Code review comment for lp:~camptocamp/banking-addons/bank-statement-reconcile-7.0-split-sale-lep

  1. bank-statement-reconcile-7.0-split-sale-lep
  2. Merge into bank-statement-reconcile-70
Revision history for this message
Leonardo Pistone (lepistone) wrote :

Thanks Joël for your review.

I added a YAML test and fixed a bug that the test showed.

The test passes if the following two MP are merged:

* https://code.launchpad.net/~camptocamp/banking-addons/bank-statement-reconcile-7.0-fix-onchange-lep/+merge/199518
* https://code.launchpad.net/~camptocamp/banking-addons/bank-statement-reconcile-7.0-fix-yaml-related-lep/+merge/199124

They are approved already, so that should be no problem.

« Back to merge proposal