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
« Back to merge proposal
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