Review: Needs Information

1) give me a use case where your patch is fixing something for lines 16-22. As 
i said, i don't find any problem...
2) don't forget to consider my above remarks whenever you want to resubmit your 
merge proposal.
3) you should have done 1 branch per bugfix so lines 35-41 should have been 
done in another merge prop. Nevermind now, but don't forget next time.
-- 
https://code.launchpad.net/~openerp-dev/openobject-addons/trunk-bug-807961-dbr/+merge/68664
Your team OpenERP R&D Team is subscribed to branch 
lp:~openerp-dev/openobject-addons/trunk-bug-807961-dbr.

_______________________________________________
Mailing list: https://launchpad.net/~openerp-dev-gtk
Post to     : [email protected]
Unsubscribe : https://launchpad.net/~openerp-dev-gtk
More help   : https://help.launchpad.net/ListHelp

Reply via email to