Skip to content

Conversation

@arpi-odoo
Copy link

No description provided.

@robodoo
Copy link

robodoo commented Nov 3, 2025

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

Made the new contract button invisible when no contract start date

Task 5231718
1.  Go to **Time Off > Configuration > Time Off Types**.
2.  Open a type and click the "Time Off" smart button.
3.  Click **New**.
4.  An error appears.

[[Reference Video](https://drive.google.com/file/d/1vi_apL24Km5tSomQtwcTArqqWvrBR5ni/view)]

In the `_search_virtual_remaining_leaves` method, the code tries to use an operator (like `>`) to compare the `virtual_remaining_leaves`.

This operator requires **two** arguments to work. However, the code was only passing **one** argument, missing the `value` to compare against. This caused the "expects 2 arguments" error.

Passed the missing `value` to the operator call so the comparison can be completed correctly.

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

4 participants