Review: Needs Information

Slight preference for setting image to NULL in that case. Otherwise, looks 
alright. Have you tried finding out what encoding the image is in, to see if 
you could recover the images? That is, if the images are rendered properly in 
the interface for those employees. Maybe you could post an example string in 
that case?


-- 
https://code.launchpad.net/~savoirfairelinux-openerp/openupgrade-addons/hr_employee_image/+merge/192728
Your team OpenUpgrade Committers is subscribed to branch lp:openupgrade-addons.

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

Reply via email to