Skip to content

[IMP] empty#5011

Open
arpi-odoo wants to merge 3 commits intomaster-hr-onboardingfrom
master-hr-onboarding-moali
Open

[IMP] empty#5011
arpi-odoo wants to merge 3 commits intomaster-hr-onboardingfrom
master-hr-onboarding-moali

Conversation

@arpi-odoo
Copy link

No description provided.

@robodoo
Copy link

robodoo commented Feb 2, 2026

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

Mohamed-moali added 2 commits February 3, 2026 13:44
The new contract option under the hr employee payroll tab will be conditionally visible in case the employee already had a contract. Otherwise the user is not permitted to initiate a new contract without an older one

Task: 5904916
…ction

Steps to reproduce:
1- Setup a leave type to 'require allocation' = True.
2- Create allocations to an employee of the same leave type.
3- Create a leave from the timeoff smart button on the leave type list view.
4- Create a leave for the allocated employee and select the leave type field.
5- System throws an exception.

**Issue**:
Due to an error in the virtual_remaining_leaves computation function, leave types that required allocation were throwing an error to the user whenever they tried assigning time off to any employee due to a missing parameter in the method being called '_compute_virtual_remaining_leave'

**Fix**:
Added the value of the number of days requested for the leave in the 'is_valid' function in order to satisfy the operands requirements.

Task: 5905492
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