← Back to team overview

openerp-dev-web team mailing list archive

Re: [Merge] lp:~openerp-dev/openobject-addons/trunk-bug-700928-ksa into lp:openobject-addons

 

Review: Needs Fixing
1) Why the name is lot_id, if the field is of type boolen?
it makes sense that its a many2one field, improve the name so it may speak its purpose from its name!
and this is for our internal purposes, its good to put it as readonly and invisible,
2) no need to use track_prod variable, just use the value directly..
-- 
https://code.launchpad.net/~openerp-dev/openobject-addons/trunk-bug-700928-ksa/+merge/54691
Your team OpenERP R&D Team is subscribed to branch lp:~openerp-dev/openobject-addons/trunk-bug-700928-ksa.



References