Skip to content

[IMP] empty#5184

Open
arpi-odoo wants to merge 3 commits into
master-hr-onboardingfrom
master-hr-onboarding-chean
Open

[IMP] empty#5184
arpi-odoo wants to merge 3 commits into
master-hr-onboardingfrom
master-hr-onboarding-chean

Conversation

@arpi-odoo

Copy link
Copy Markdown

No description provided.

@robodoo

robodoo commented Jun 23, 2026

Copy link
Copy Markdown

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

Steps to reproduce:

- Open an employee without a contract start date.
- Check the contract section.

Cause:
The "New Contract" button was displayed even when no contract start date was defined.

Solution:
Add an invisible condition on the button to hide it when contract_date_start is not set.

Task: 6326571
Step to reproduce:
Search on the virtual_remaining_leaves field using a comparison
operator such as '>'.

Cause:
The search method called the comparison operator with only the
field value, while operators such as operator.gt expect both the
field value and the domain value.

Solution:
Pass the domain value to the comparison operator and add a test
covering a search on virtual_remaining_leaves with an allocated
leave type.

Task: 6327165
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.

3 participants