Review: Approve

Hello,

Code seems correct; I just added a dict access protection. I keep the branch 
alive, because we might want to add features in social-account before doing the 
final merge.

I just have two questions about the specifications:
1/ no need_action ?
2/ no notification when changing currency ?

Best regards,

Thibault.
-- 
https://code.launchpad.net/~openerp-dev/openobject-addons/trunk-social-account-rga/+merge/98341
Your team OpenERP R&D Team is subscribed to branch 
lp:~openerp-dev/openobject-addons/trunk-social-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