openerp-community-reviewer team mailing list archive
-
openerp-community-reviewer team
-
Mailing list archive
-
Message #06989
Re: [Merge] lp:~camptocamp/account-financial-tools/add-manual-line-and-fees-nbi into lp:account-financial-tools
Review: Approve code review
Last few remarks to the end of the diff:
- 1407 the line is perhaps unintentional?
- 1299 typo "initialize"
- 2343 typo bypasses.
These are just small doc typos, so I'm approving the MP anyway.
For this kind of small typos, I'd not insist too much, since it will be so much easier to fix those on github, editing and merging from the web interface.
Thanks!
--
https://code.launchpad.net/~camptocamp/account-financial-tools/add-manual-line-and-fees-nbi/+merge/218584
Your team OpenERP Community Reviewer/Maintainer is subscribed to branch lp:account-financial-tools.
References