banking-addons-team team mailing list archive

["Leonardo Pistone @ camptocamp" <leonardo.pistone@xxxxxxxxxxxxxx>]

The proposal to merge lp:~camptocamp/banking-addons/bank-statement-reconcile-7.0-add-tests-no-split-lep into lp:banking-addons/bank-statement-reconcile-7.0 has been updated.

Description changed to:

Tests for the autocompletion of banking statements.

* Customer invoice from CI number
* Supplier invoice from SI number
* Partner name
* Partner code
* Sale Order from the Transaction ID

In fact 1 of the tests doesn't pass: the one based on Partner Name. It looks to me that the test is correct: it has a statement line that includes the name Vauxoo, and fails to match the partner Vauxoo because, as it seems, it searches the whole line into the partner name and not vice versa.

I propose that anyway to give it some visibility. I'm sure we can get it right in this proposal.

Related branches:
* This has a similar test that passes based on SO numbers. https://code.launchpad.net/~camptocamp/banking-addons/bank-statement-reconcile-7.0-split-sale-lep/+merge/197544

* Two branches needed to fix bugs that came out here: https://code.launchpad.net/~camptocamp/banking-addons/bank-statement-reconcile-7.0-fix-yaml-related-lep/+merge/199124 and https://code.launchpad.net/~camptocamp/banking-addons/bank-statement-reconcile-7.0-fix-onchange-lep/+merge/199518

Thanks!

For more details, see:
https://code.launchpad.net/~camptocamp/banking-addons/bank-statement-reconcile-7.0-add-tests-no-split-lep/+merge/199835
-- 
https://code.launchpad.net/~camptocamp/banking-addons/bank-statement-reconcile-7.0-add-tests-no-split-lep/+merge/199835
Your team Banking Addons Core Editors is requested to review the proposed merge of lp:~camptocamp/banking-addons/bank-statement-reconcile-7.0-add-tests-no-split-lep into lp:banking-addons/bank-statement-reconcile-7.0.