Skip to content

[FIX] hr_payroll_account: restrict invalid days in accrual plans and milestones#5178

Merged
arpi-odoo merged 1 commit into
master-hr-onboarding-maelkfrom
master-hr-onboarding-accrual_ux-maelk
Jun 18, 2026
Merged

[FIX] hr_payroll_account: restrict invalid days in accrual plans and milestones#5178
arpi-odoo merged 1 commit into
master-hr-onboarding-maelkfrom
master-hr-onboarding-accrual_ux-maelk

Conversation

@manalelk

Copy link
Copy Markdown

Currently, it's possible to select non-existent days like February 31st when configuring accrual plans and their milestones.

Restricts the day selection according to the month.

Task-6297384

@robodoo

robodoo commented Jun 15, 2026

Copy link
Copy Markdown

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

@manalelk manalelk force-pushed the master-hr-onboarding-accrual_ux-maelk branch from 14eb98b to de3dbcb Compare June 15, 2026 12:32
@manalelk manalelk force-pushed the master-hr-onboarding-maelk branch from f2d6c7e to 0c8eba6 Compare June 17, 2026 13:59
@manalelk manalelk force-pushed the master-hr-onboarding-accrual_ux-maelk branch 2 times, most recently from 4ce37b3 to fe36bdb Compare June 17, 2026 15:32
Comment thread addons/hr_holidays/static/src/components/accrual_level/accrual_levels.scss Outdated
…milestones

Currently, it is possible to select non-existent days like February 31st
when configuring accrual plans and their milestones.

Restricts the day selection according to the month.

Task-6297384
@manalelk manalelk force-pushed the master-hr-onboarding-accrual_ux-maelk branch from fe36bdb to 3a7b26a Compare June 18, 2026 07:17
@arpi-odoo arpi-odoo merged commit 6b435cb into master-hr-onboarding-maelk Jun 18, 2026
@arpi-odoo arpi-odoo deleted the master-hr-onboarding-accrual_ux-maelk branch June 18, 2026 07:39
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