Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

TA#72215 [16.0] [MIG] timesheet_holidays_time_control_binding #54

Open
wants to merge 2 commits into
base: 16.0
Choose a base branch
from

Conversation

rivo2302
Copy link

No description provided.

@lanto-razafindrabe lanto-razafindrabe changed the title Ta#72215 [16.0] [MIG] timesheet_holidays_time_control_binding TA#72215 [16.0] [MIG] timesheet_holidays_time_control_binding Jan 15, 2025
class HrLeave(models.Model):
_inherit = "hr.leave"

def action_validate(self):
Copy link
Contributor

@lanto-razafindrabe lanto-razafindrabe Jan 15, 2025

Choose a reason for hiding this comment

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

image
Hello @rivo2302
I have tested locally the module, I got this User Error of Odoo when trying to completely approving the time off. Odoo added a check in v16 that not allowed us to update created timesheet linked to a time off.

I have found unit test about it also failed.

And when testing locally, could you update screenshots please (not in debug mode) ? Thanks

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants