Skip to content

16.0 mig hr shift#35

Open
hitrosol wants to merge 14 commits into
OCA:16.0from
solusiaglis:16.0-mig-hr_shift
Open

16.0 mig hr shift#35
hitrosol wants to merge 14 commits into
OCA:16.0from
solusiaglis:16.0-mig-hr_shift

Conversation

@hitrosol
Copy link
Copy Markdown

Standard migration...

chienandalu and others added 13 commits April 19, 2026 09:09
TT50623

Co-authored-by: chienandalu

[UPD] Update hr_shift.pot

[BOT] post-merge updates
Changes done:
- Add rules related to shift lines
- Add My shifts filter + group by fields
- Add template + employee fields to calendar view
- Set widget="many2one_avatar_employee" to employee_id field (tree view)
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate.

Translation: shift-planning-15.0/shift-planning-15.0-hr_shift
Translate-URL: https://translation.odoo-community.org/projects/shift-planning-15-0/shift-planning-15-0-hr_shift/
Use case example:
- Create an hr.shift.planning.line record with a start time of 00:00:00 and an end time of 08:00:00
- The employee must have a work schedule after 08:00:00 (for example, starting at 13:00:00)
- The report from the hr_attendance_report_theoretical_time module is used. The hr_shift methods will mix records and cause an error.

TypeError: cannot union different models: 'hr.shift.planning.line()' and 'resource.calendar.attendance()'

TT60298
This module replaces theoretical calendar attendances by the shift ones
in a conservative way, only removing those that belong to the exact
shift. That way, if no shift is specified, the usual calendar will
apply.

But when the shift is at or around midnight, there can be 2 dates
involved, the starting day, and the ending day. If the calendar assigned
to the employee has attendance entries in both of them, the replacing
of them won't be correct.

Thus, we should remove all the attendances entries belonging to any of
the days of the shift, no matter the hours.

TT61053
@hitrosol hitrosol force-pushed the 16.0-mig-hr_shift branch from 788317a to 3491092 Compare April 20, 2026 02:03
@hitrosol
Copy link
Copy Markdown
Author

I'm not sure why the pre-commit fail

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants