← Back to team overview

openerp-community-reviewer team mailing list archive

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