← Back to team overview

savoirfairelinux-openerp team mailing list archive

Re: lp:~savoirfairelinux-openerp/openerp-travel/airport_airline_railwaystation_travel into lp:~savoirfairelinux-openerp/openerp-travel/travel

 

Review: Needs Fixing code review, test

l.7 comma missing
l.27 Inconsistency in capitalization
l.26,61,100 No need for separators if you have groups, just use <group string=...>, I don't think such big headers (groups and separators) are necessary when these check boxes are in the same notebook page
l.110: this file is called partner_view when the rest are called res_partner_view, please bzr move it
-- 
https://code.launchpad.net/~savoirfairelinux-openerp/openerp-travel/airport_airline_railwaystation_travel/+merge/197106
Your team Savoir-faire Linux' OpenERP is subscribed to branch lp:~savoirfairelinux-openerp/openerp-travel/travel.


References