Code review comment for lp:~bcim/account-invoicing/7.0-account_invoice_line_default_account

Hi, Jacques,

>I disagree with that as no fields in the accounting tab are displayed/hidden based on the customer/supplier.

This is why I told you that, because it would be desirable to hide "property_account_expense" and "auto_update_account_expense" fields when partner is not a supplier, because that fields don't apply. This can be done with attributes property testing "customer" field.

>Can you point me out where is this documented? I didn't found any example nor any mention in the contribute documentation.

Indeed, this is not documented yet (I mentioned about the necessity of contribution/author documentation in the draft pad, but no work has been done in this way). However, there are some comments in MPs that points to this and it's also mentioned in new apps website thread on the community list. The goal for this kind of representation is to get visibility of contributions on the app website in the future, but it's not mandatory.

Regards