Code review comment for lp:~initos.com/openerp-hr/7.0-change-public-holiday

Revision history for this message
Leonardo Pistone (lepistone) wrote :

Dear Markus, Nikolina,

for "empty countries" meaning "all countries" it's ok for me, if you can say that on the help string of the field, for example. That way I agree with you migrations are not an issue.

As for the interactions other modules, I am not sure, I am abstaining so that others can speak.

Thanks!

review: Abstain

« Back to merge proposal