Code review comment for lp:~jean-lelievre/account-payment/account_prepayment

Revision history for this message
Maxime Chambreuil (http://www.savoirfairelinux.com) (max3903) wrote :

Hello Jean,

l117: you need an extra line to be PEP8 compliant. Running flake8 on your module will help you with that.

You made the translation in chinese but the pot file for other translations is missing.

Otherwise LGTM.

Thanks!

review: Needs Fixing (code review)

« Back to merge proposal