-
Notifications
You must be signed in to change notification settings - Fork 23.1k
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
[IMP] hr_attendance: Add overtime reporting #165084
[IMP] hr_attendance: Add overtime reporting #165084
Conversation
69298da
to
bc17efd
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.
Minor comment 🙌
</record> | ||
|
||
<menuitem id="menu_hr_attendance_overtime_reporting" name="Extra Hours" parent="menu_hr_attendance_reporting" sequence="15" groups="hr_attendance.group_hr_attendance_officer" action="hr_attendance_overtime_action"/> |
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 my personal opinion, but I think it's better to put menuitems together.. so move this to addons/hr_attendance/views/hr_attendance_view.xml
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.
hmmm, if there was a menuitems.xml file why not, but here i prefer to have it right next to the action 🤷♂️
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.
LGTM just a small comment about timedelta :)
task-3894091
bc17efd
to
96a1553
Compare
<menuitem id="menu_hr_attendance_reporting" name="Reporting" parent="menu_hr_attendance_root" sequence="15" groups="hr_attendance.group_hr_attendance_officer"/> | ||
|
||
<menuitem id="menu_hr_attendance_attendance_reporting" name="Worked Hours" parent="menu_hr_attendance_reporting" sequence="15" groups="hr_attendance.group_hr_attendance_officer" action="hr_attendance_reporting"/> |
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.
Line length 😇
<menuitem id="menu_hr_attendance_reporting" name="Reporting" parent="menu_hr_attendance_root" sequence="15" groups="hr_attendance.group_hr_attendance_officer"/> | |
<menuitem id="menu_hr_attendance_attendance_reporting" name="Worked Hours" parent="menu_hr_attendance_reporting" sequence="15" groups="hr_attendance.group_hr_attendance_officer" action="hr_attendance_reporting"/> | |
<menuitem id="menu_hr_attendance_reporting" name="Reporting" | |
parent="menu_hr_attendance_root" sequence="15" | |
groups="hr_attendance.group_hr_attendance_officer"/> | |
<menuitem id="menu_hr_attendance_attendance_reporting" name="Worked Hours" | |
parent="menu_hr_attendance_reporting" sequence="15" | |
groups="hr_attendance.group_hr_attendance_officer" action="hr_attendance_reporting"/> |
</record> | ||
|
||
<menuitem id="menu_hr_attendance_overtime_reporting" name="Extra Hours" parent="menu_hr_attendance_reporting" sequence="15" groups="hr_attendance.group_hr_attendance_officer" action="hr_attendance_overtime_action"/> |
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.
same here
<menuitem id="menu_hr_attendance_overtime_reporting" name="Extra Hours" parent="menu_hr_attendance_reporting" sequence="15" groups="hr_attendance.group_hr_attendance_officer" action="hr_attendance_overtime_action"/> | |
<menuitem id="menu_hr_attendance_overtime_reporting" name="Extra Hours" | |
parent="menu_hr_attendance_reporting" sequence="15" | |
groups="hr_attendance.group_hr_attendance_officer" action="hr_attendance_overtime_action"/> |
robodoo r+ |
task-3894091 closes #165084 Signed-off-by: Bertrand Dossogne (bedo) <bedo@odoo.com>
task-3894091 closes odoo#165084 Signed-off-by: Bertrand Dossogne (bedo) <bedo@odoo.com>
task-3894091