Review: Approve

Works for me, though the URL building might be clearer via _.str.sprintf. And 
I'm not sure there's a need to use self.session.prefix.
-- 
https://code.launchpad.net/~openerp-dev/openerp-web/trunk-user_img-tde/+merge/100081
Your team OpenERP R&D Team is subscribed to branch 
lp:~openerp-dev/openerp-web/trunk-user_img-tde.

_______________________________________________
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