Code review comment for lp://staging/~sbi/account-invoicing/6.1-account_invoice_total

Revision history for this message
Niels Huylebroeck (red15) wrote :

What is the purpose of the README.md file ? Is this some form of markup ? If so we should as community gather around a central format which everyone can agree upon so we can make such files a requirement when committing new modules.

Also a bit strange why required="2" on the check_total field ? Why is value 2 not 1 or True, I prefer the latter personally.

« Back to merge proposal