Review: Needs Fixing Hello,
Here are my remarks ! Concerning the needaction mechanism, we should not use it that way in procurements. - remove reference to ir.needaction_mixin and the get_needaction_user_ids method. This is not necessary for procurements. - remove the added user_id field Some remarks on messages: - "Procurement has been set to <b>running</b> state." -> "Procurement has been set to <b>running</b>." - "Procurement has been set to <b>ready</b> state." -> "Procurement has been set to <b>ready</b>." -- https://code.launchpad.net/~openerp-dev/openobject-addons/trunk-social-procurement-tpa/+merge/103409 Your team OpenERP R&D Team is subscribed to branch lp:~openerp-dev/openobject-addons/trunk-openchatter. _______________________________________________ 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

