← Back to team overview

banking-addons-team team mailing list archive

Thread
Date

Re: [Merge] lp:~therp-nl/banking-addons/ba70-lp1296789-remove_static into lp:banking-addons

To: mp+212502@xxxxxxxxxxxxxxxxxx
From: Pedro Manuel Baeza <pedro.baeza@xxxxxxxxx>
Date: Thu, 27 Mar 2014 17:41:06 -0000
In-reply-to: <20140324195921.25772.88498.launchpad@ackee.canonical.com>
Reply-to: mp+212502@xxxxxxxxxxxxxxxxxx
Sender: bounces@xxxxxxxxxxxxx

Review: Approve code review

I see current method of adding parsers is a little twisted, but if there is no other solution, go ahead with the patch.

Have you seen method used on account_statement_base_import?

Regards.
-- 
https://code.launchpad.net/~therp-nl/banking-addons/ba70-lp1296789-remove_static/+merge/212502
Your team Banking Addons Core Editors is subscribed to branch lp:banking-addons.

References

[Merge] lp:~therp-nl/banking-addons/ba70-lp1296789-remove_static into lp:banking-addons
From: Stefan Rijnhart (Therp), 2014-03-24