Skip to content

[18.0][ADD] hr_work_entry_timesheet_validate#1525

Open
remi-filament wants to merge 2 commits intoOCA:18.0from
lefilament:18.0-add-hr_work_entry_timesheet_validate
Open

[18.0][ADD] hr_work_entry_timesheet_validate#1525
remi-filament wants to merge 2 commits intoOCA:18.0from
lefilament:18.0-add-hr_work_entry_timesheet_validate

Conversation

@remi-filament
Copy link
Contributor

Glue module between hr_work_entry_validate ( #1524 ) and hr_work_entry_timesheet ( #1523 ) to show timesheet on work entry validation interface for employees.

Copy link
Contributor

@alexey-pelykh alexey-pelykh left a comment

Choose a reason for hiding this comment

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

Thanks for this. The glue approach between hr_work_entry_timesheet and hr_work_entry_validate makes sense.

Tests pass, but the "Detect unreleased dependencies" check is failing — this depends on modules that haven't been merged into the 18.0 branch yet. Once the dependencies land, this should be good to go.

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