Code review comment for lp:~savoirfairelinux-openerp/openerp-hr/department_sequence_concatination_NoneType
Revision history for this message
| Katja Matthes (katja-matthes) wrote | # |
review:
Approve
(code review)
© 2004
Canonical Ltd.
•
Terms of use
•
Data privacy
•
Contact Launchpad Support
•
Blog
•
Careers
•
System status
•
0e1f616
(Get the code!)
Hello,
looks good. Tests run fine.
I think, since you made a fix you should update the module's version number.
Regards.