Skip to content

[FIX] hr_holidays: Traceback on Time Off Type#5154

Open
raame-odoo wants to merge 1 commit intomaster-hr-onboarding-raamefrom
master-hr-onboarding-traceback_timeoff_type-raame
Open

[FIX] hr_holidays: Traceback on Time Off Type#5154
raame-odoo wants to merge 1 commit intomaster-hr-onboarding-raamefrom
master-hr-onboarding-traceback_timeoff_type-raame

Conversation

@raame-odoo
Copy link
Copy Markdown

@raame-odoo raame-odoo commented May 7, 2026

When navigating (Time Off >> Configuration >> Time Off Types >> Open any type, navigate to the Smart button Time Off >> Click on New >> Time Off Type), it leads to an error. The nested function is_valid in function _search_virtual_remaining_leaves in hr_work_entry_type uses an operator in it's return statement that has on parameter when it should receive two parameters. the second parameter "value" was added.

Task: 6197121

@robodoo
Copy link
Copy Markdown

robodoo commented May 7, 2026

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

@raame-odoo raame-odoo force-pushed the master-hr-onboarding-traceback_timeoff_type-raame branch from 1ca061b to 32a245b Compare May 7, 2026 08:27
When navigating (Time Off >> Configuration >> Time Off Types >> Open any type, navigate to the Smart button Time Off >> Click on New >> Time Off Type), it leads to an error. The nested function is_valid in function _search_virtual_remaining_leaves in hr_work_entry_type uses an operator in it's return statement that has on parameter when it should receive two parameters. the second parameter "value" was added.

Task: 6197121
@raame-odoo raame-odoo force-pushed the master-hr-onboarding-traceback_timeoff_type-raame branch from 32a245b to d886614 Compare May 7, 2026 08:37
@raame-odoo raame-odoo changed the title TEST [FIX] hr_holidays: Traceback on Time Off Type May 7, 2026
Copy link
Copy Markdown

@karygauss03 karygauss03 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, good job 💯

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