Code review comment for lp:~camptocamp/account-financial-tools/add-manual-line-and-fees-nbi

Revision history for this message
Romain Deheele - Camptocamp (romaindeheele) wrote :

Hello,

Just little things as:
- I see deprecated attribute type on form views (l.869, 906)
- l.2221: s/Euro policy level/Usd policy level
- l.1310, 1311: '==' instead of '='

Otherwise, few little typos:
- l.1709: s/writen/written
- l.1709: s/fields/field
- l.2199: s/Initialaize/Initialize
- I would add commas on l.280, 290, 1392
- I would remove double spaces on l.699, 725

thanks for these new features,

Romain

review: Needs Fixing (code review)

« Back to merge proposal