← Back to team overview

banking-addons-team team mailing list archive

Re: lp:~ruchir.shukla/banking-addons/account_banking_nl_mt940structured into lp:banking-addons

 

Review: Needs Information

On the contrary, I am dubious about using the same parser for various Dutch banks. But I do not have any examples ready. Of course, even then we would be seriously blessed by a common base that could be inherited by bank specific modules (note the large code base shared between patu, HBSC and this module). Would it be possible to refactor so that for instance HBSC could inherit from this module?

Do I read correctly that the remote account is not retrieved by this parser? Parsing of tag 86 is pretty sparse, aming towards the unstructured tags. Don't you encounter a lot of structured tags 86 nowadays?

-- 
https://code.launchpad.net/~ruchir.shukla/banking-addons/account_banking_nl_mt940structured/+merge/172233
Your team Banking Addons Core Editors is subscribed to branch lp:banking-addons.


References