Skip to content

[IMP] empty#5135

Draft
lebm-odoo wants to merge 3 commits into
master-hr-onboardingfrom
master-hr-onboarding-belam
Draft

[IMP] empty#5135
lebm-odoo wants to merge 3 commits into
master-hr-onboardingfrom
master-hr-onboarding-belam

Conversation

@lebm-odoo
Copy link
Copy Markdown

No description provided.

@robodoo
Copy link
Copy Markdown

robodoo commented May 4, 2026

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

belam-odoo added 2 commits May 8, 2026 14:55
In the employee view > payroll_informations :

BEFORE:
The button "New Contract" was displayed unconditionnally
AFTER:
Since it makes no sense to start a contract without a start date, that button will only appear when the start date is defined

task-6186027
Bug introduced:

Introduced by commit: 6485081

---
Step to reproduce:
1. Make sure the employee has at least one time off allocation (work_entry with requires_allocation set to True).

In-app : TIME OFF > Management > Allocations > New...

2. Create a time off for that employee and try to select the Time Type

In-app: TIME OFF > Management > Time Off > New > Select employee + Select Time Type

Reason:
Syntax error - one argument was passed to a method instead of two
The method was only called by an domain RPC search in the form.

Solution:
Added the second parameters in the method call
---

Task #6187725
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