Code review comment for lp:~savoirfairelinux-openerp/openerp-hr/unique_code

Revision history for this message
Sandy Carter (http://www.savoirfairelinux.com) (sandy-carter) wrote :

Thank you for your review, Daniel,

The .pot file is already included, I think you missed it.

What's wrong with an optional custom name_get, it will only display the code if you explicitly say so in the context.
Is it really necessary to divide this module even further. Seems that if you use code for departments, you would want to list them as codes.

« Back to merge proposal