Elabore/hr-tools
9
0
Fork 0
Code Issues Pull Requests Actions Packages Releases Activity

18.0-migrate-hr_holidays_timeoff_analysis-coming-from-repo-elabore-addons #12

Closed
mondot wants to merge 3 commits from 18.0-migrate-hr_holidays_timeoff_analysis-coming-from-repo-elabore-addons into 18.0
pull from: 18.0-migrate-hr_holidays_timeoff_analysis-coming-from-repo-elabore-addons
merge into: Elabore:18.0
Conversation 3 Commits 3 Files Changed 16 +1649 -5
mondot commented 2025-10-09 12:46:39 +00:00
Owner
Copy Link
No description provided.
mondot added 3 commits 2025-10-09 12:46:39 +00:00
[ADD] copy paste module hr_holidays_timeoff_analysis from version 16.0 (from repo elabore-addons) 37a8bda090
[IMP] hr_holidays_timeoff_analysis : apply changes proposed by pre-commit 6135fa68ee
[IMP] hr_holidays_timeoff_analysis : migrate from version 16 to 18
Some checks failed
pre-commit / pre-commit (pull_request) Failing after 1m27s
Details
d3c47266b6
mondot added 2 commits 2025-10-10 09:28:46 +00:00
[IMP] hr_holidays_timeoff_analysis : migrate from version 16 to 18
Some checks failed
pre-commit / pre-commit (pull_request) Failing after 1m27s
Details
d3c47266b6
[IMP] hr_holidays_timeoff_analysis : migrate from version 16 to 18
Some checks failed
pre-commit / pre-commit (pull_request) Failing after 1m25s
Details
8c8b617e8c
mondot added 2 commits 2025-10-10 09:35:46 +00:00
[IMP] hr_holidays_timeoff_analysis : migrate from version 16 to 18
Some checks failed
pre-commit / pre-commit (pull_request) Failing after 1m25s
Details
8c8b617e8c
[IMP] hr_holidays_timeoff_analysis : migrate from version 16 to 18
All checks were successful
pre-commit / pre-commit (pull_request) Successful in 1m25s
Details
d24477bed9
mondot added 2 commits 2025-10-10 10:18:52 +00:00
[IMP] hr_holidays_timeoff_analysis : migrate from version 16 to 18
All checks were successful
pre-commit / pre-commit (pull_request) Successful in 1m25s
Details
d24477bed9
[IMP] hr_holidays_timeoff_analysis : migrate from version 16 to 18
All checks were successful
pre-commit / pre-commit (pull_request) Successful in 1m26s
Details
bc64030847
mondot commented 2025-10-10 10:22:01 +00:00
Author
Owner
Copy Link

Cc @laetitiadacosta

j'ai maj les tests pour qu'ils passent. Par contre il y en a un qui fail avec cette erreur

2025-10-10 10:18:12,299 134 ERROR odoo_test_hr_holidays odoo.addons.hr_holidays_timeoff_analysis.tests.test_hr_leave_timeoff_day: ERROR: TestHrLeaveTimeoffDay.test_leave_duration_by_day_hour_compare_to_employee_calendar
Traceback (most recent call last):
  File "/opt/odoo/auto/dev/hr-tools/hr_holidays_timeoff_analysis/tests/test_hr_leave_timeoff_day.py", line 229, in test_leave_duration_by_day_hour_compare_to_employee_calendar
    leave._compute_number_of_hours_display()
AttributeError: 'hr.leave' object has no attribute '_compute_number_of_hours_display'

Et effectivement dans la version 18, la fonction _compute_number_of_hours_display n'existe plus pour le modèle hr.leave.

Il va falloir que tu vois comment modifier la fonction

    def compute_leave_duration_by_day(self, leave):
        """
        Compute the leave duration by day based on the leave type.
        """
        [...]
        elif leave.request_unit_hours:
            leave_duration_by_day = leave.number_of_days_display
        [...]

pour l'adapter en v18

Cc @laetitiadacosta j'ai maj les tests pour qu'ils passent. Par contre il y en a un qui fail avec cette erreur ``` 2025-10-10 10:18:12,299 134 ERROR odoo_test_hr_holidays odoo.addons.hr_holidays_timeoff_analysis.tests.test_hr_leave_timeoff_day: ERROR: TestHrLeaveTimeoffDay.test_leave_duration_by_day_hour_compare_to_employee_calendar Traceback (most recent call last): File "/opt/odoo/auto/dev/hr-tools/hr_holidays_timeoff_analysis/tests/test_hr_leave_timeoff_day.py", line 229, in test_leave_duration_by_day_hour_compare_to_employee_calendar leave._compute_number_of_hours_display() AttributeError: 'hr.leave' object has no attribute '_compute_number_of_hours_display' ``` Et effectivement dans la version 18, la fonction _compute_number_of_hours_display n'existe plus pour le modèle `hr.leave`. Il va falloir que tu vois comment modifier la fonction ``` def compute_leave_duration_by_day(self, leave): """ Compute the leave duration by day based on the leave type. """ [...] elif leave.request_unit_hours: leave_duration_by_day = leave.number_of_days_display [...] ``` pour l'adapter en v18
laetitiadacosta approved these changes 2025-10-13 08:43:57 +00:00
mondot requested review from laetitiadacosta 2025-10-15 08:26:20 +00:00
mondot self-assigned this 2025-11-18 11:35:20 +00:00
mondot commented 2025-12-18 09:02:57 +00:00
Author
Owner
Copy Link

On ferme car le module a beaucoup évolué en v16

On ferme car le module a beaucoup évolué en v16
mondot closed this pull request 2025-12-18 09:02:57 +00:00
mondot deleted branch 18.0-migrate-hr_holidays_timeoff_analysis-coming-from-repo-elabore-addons 2025-12-18 09:03:02 +00:00
All checks were successful
pre-commit / pre-commit (pull_request) Successful in 1m26s
Details

Pull request closed

This pull request cannot be reopened because the branch was deleted.
Sign in to join this conversation.
Reviewers
No Reviewers
laetitiadacosta
Labels
Clear labels
bug
Something isn't working
documentation
Improvements or additions to documentation
duplicate
This issue or pull request already exists
enhancement
New feature or request
good first issue
Good for newcomers
help wanted
Extra attention is needed
invalid
This doesn't seem right
question
Further information is requested
wontfix
This will not be worked on
No Label
Milestone
No items
No Milestone
Assignees
Clear assignees
admin borisgallet elabore_bot elabore_vps laetitiadacosta mathieusarrasin mondot stephansainleger vlab
No Assignees mondot
2 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: Elabore/hr-tools#12
No description provided.
Delete Branch "18.0-migrate-hr_holidays_timeoff_analysis-coming-from-repo-elabore-addons"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?