← Back to team overview

banking-addons-team team mailing list archive

Re: [Merge] lp:~therp-nl/banking-addons/ba70-lp1295163-refactor_period_lookup into lp:banking-addons

 

Review: Needs Information

+        except except_osv:
59	+            if date:
60	+                raise

This is something not needed . 
as there date field is already required field for statement and statement lines .
Can you specify in which condition it will not have value date variable?
-- 
https://code.launchpad.net/~therp-nl/banking-addons/ba70-lp1295163-refactor_period_lookup/+merge/211970
Your team Banking Addons Core Editors is subscribed to branch lp:banking-addons.


References