Review: Resubmit

> For the record, a log of what we just discussed live:

All done except :

> - l.42,48: use browse for readability

kept one occurence

> - l.50,51: look for domain builder methods in osv.expression, e.g.
> expression.AND expression.OR

didn't used it

> - l.61,62: type = 'field' is missing in WHERE clauses

used type='model' as we discussed

-- 
https://code.launchpad.net/~openerp-dev/openobject-server/trunk-scrap-translatedialog-fme/+merge/122641
Your team OpenERP R&D Team is subscribed to branch 
lp:~openerp-dev/openobject-server/trunk-scrap-translatedialog-fme.

_______________________________________________
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