Code review comment for lp://qastaging/~daniel-watkins-credativ/poweremail/fix-server-actions

Revision history for this message
Niels Huylebroeck (red15) wrote :

Just goes to show that openerp can sometimes really think backwards, why didn't they just say

attrs="{'required': [('state','=','copy_object')]}"

But I tested it and this fix works too :)

review: Approve

« Back to merge proposal