-
-
Notifications
You must be signed in to change notification settings - Fork 664
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[MIG][11.0] hr attendance reason #587
Conversation
6584ff4
to
05a9b5f
Compare
name = fields.Char( | ||
String='Reason', | ||
help='Specifies the reason leaving soon or arriving late', | ||
required=True, index=True) | ||
code = fields.Char('Reason Code') | ||
action_type = fields.Selection( | ||
[('sign_in', 'Sign in'), |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Do not change the name without a migration script please.
You can make the script or keep the old name.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Okay, I will keep sign in and sign out then.
hr_attendance_reason/i18n/es.po
Outdated
@@ -0,0 +1,129 @@ | |||
# Translation of Odoo Server. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It's better to maintain the translation directly on weblate
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
OK!
hr_attendance_reason/README.rst
Outdated
.. |badge3| image:: https://img.shields.io/badge/github-OCA%2Fhr-lightgray.png?logo=github | ||
:target: https://github.com/OCA/hr/tree/10.0/hr_attendance_reason | ||
:alt: OCA/hr | ||
.. |badge3| image:: https://img.shields.io/badge/github-OCA%2Fhr_attendance_reason-lightgray.png?logo=github |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Could you split the readme with the new standard model please? 😉
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Sorry, it is splitted. I didn't notice it
05a9b5f
to
23f2665
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
code review: lgtm
maybe some aesthetic changes (optional)
82249af
to
e858e3e
Compare
e858e3e
to
de4c6b5
Compare
Migration of hr_attendance_reason to 11.0.