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-fix-yaml-related-lep into lp:banking-addons/bank-statement-reconcile-7.0 has been updated.

Description changed to:

This is a workaround needed for the YAML tests in the account_statement_base_completion module.

These tests are developed here:
https://code.launchpad.net/~camptocamp/banking-addons/bank-statement-reconcile-7.0-add-tests-lep

The constraint uses a related stored field that does not work in YAML tests. Here the check is kept, but is made explicit without using the related field.

As a sidenote, if the field related is made store=False, everything works fine (on YAML tests). When entering data via the interface and not via yaml tests, everything works fine as well.

This probably needs a proper, general solution. However, IMO this workaround is not too ugly and bad and we could also keep it anyway.

Thanks!

For more details, see:
https://code.launchpad.net/~camptocamp/banking-addons/bank-statement-reconcile-7.0-fix-yaml-related-lep/+merge/199124
-- 
https://code.launchpad.net/~camptocamp/banking-addons/bank-statement-reconcile-7.0-fix-yaml-related-lep/+merge/199124
Your team Banking Addons Core Editors is subscribed to branch lp:banking-addons/bank-statement-reconcile-7.0.