Review: Approve code review

Good!

One small remark: why is the function _all_invoice_tax_line_computed not in the 
invoice class?

To show that it does not use any class or instance data, we can make it a 
@staticmethod.

I do not consider this a significant problem though, so I approve in any case.

Thanks
-- 
https://code.launchpad.net/~camptocamp/account-invoicing/7.0-add-swedish_rounding-yvr/+merge/201598
Your team Account Core Editors is subscribed to branch lp:account-invoicing.

-- 
Mailing list: https://launchpad.net/~openerp-community-reviewer
Post to     : [email protected]
Unsubscribe : https://launchpad.net/~openerp-community-reviewer
More help   : https://help.launchpad.net/ListHelp

Reply via email to