Skip to content

[FIX] hr_holidays: fix traceback on time off type#5174

Open
manalelk wants to merge 1 commit into
master-hr-onboarding-maelkfrom
master-hr-onboarding-traceback_on_time_off_type-maelk
Open

[FIX] hr_holidays: fix traceback on time off type#5174
manalelk wants to merge 1 commit into
master-hr-onboarding-maelkfrom
master-hr-onboarding-traceback_on_time_off_type-maelk

Conversation

@manalelk
Copy link
Copy Markdown

@manalelk manalelk commented Jun 4, 2026

There was an error when selecting a Time Off Type with existing allocations.

Step to reproduce:

  • Create and validate an allocation for a Time Off Type that requires allocations
  • Open this Time Off Type and create a new Time Off request from the smart button
  • In the creation popup, click on the 'Time off Type' field

Reason:
The search method for virtual_remaining_leaves was missing a required parameter

Solution:
Pass the missing value parameter when calling op()

Task-6276095

@robodoo
Copy link
Copy Markdown

robodoo commented Jun 4, 2026

This PR targets the un-managed branch odoo-dev/odoo:master-hr-onboarding-maelk, it needs to be retargeted before it can be merged.

@manalelk manalelk force-pushed the master-hr-onboarding-traceback_on_time_off_type-maelk branch from 02d5467 to b6fa7ff Compare June 4, 2026 13:50
There was an error when selecting a Time Off Type with existing allocations.

Step to reproduce:
- Create and validate an allocation for a Time Off Type that requires allocations
- Open this Time Off Type and create a new Time Off request from the smart button
- In the creation popup, click on the 'Time off Type' field

Reason:
The search method for virtual_remaining_leaves was missing a required parameter

Solution:
Pass the missing value parameter when calling op()

Task-6276095
@manalelk manalelk force-pushed the master-hr-onboarding-traceback_on_time_off_type-maelk branch from b6fa7ff to daff310 Compare June 8, 2026 13:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants