Skip to content

[FIX] hr_holidays: on yearly and bi-yearly frequencies, days are clamped to max day of the selected month - #5259

Open
feraz-odoo wants to merge 1 commit into
master-hr-onboarding-ferazfrom
master-hr-onboarding-accural_ux-feraz
Open

[FIX] hr_holidays: on yearly and bi-yearly frequencies, days are clamped to max day of the selected month#5259
feraz-odoo wants to merge 1 commit into
master-hr-onboarding-ferazfrom
master-hr-onboarding-accural_ux-feraz

Conversation

@feraz-odoo

@feraz-odoo feraz-odoo commented Aug 31, 2026

Copy link
Copy Markdown

Steps to reproduce: Possible to have impossible dates like february31 if the day was chosen after the month on yearly and bi-yearly frequencies.
Also added a test to ensure date clamping stays
task-6522773

@robodoo

robodoo commented Aug 31, 2026

Copy link
Copy Markdown

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

@feraz-odoo
feraz-odoo force-pushed the master-hr-onboarding-accural_ux-feraz branch from 9614969 to f6bbe96 Compare August 31, 2026 15:45
@feraz-odoo feraz-odoo changed the title [IMP] hr_holidays: on yearly and bi-yearly frequencies, days are clamped to max day of the selected month [FIX] hr_holidays: on yearly and bi-yearly frequencies, days are clamped to max day of the selected month Aug 31, 2026
@feraz-odoo
feraz-odoo force-pushed the master-hr-onboarding-accural_ux-feraz branch 3 times, most recently from 07aa6e8 to 3f01c80 Compare September 1, 2026 09:40
@feraz-odoo
feraz-odoo force-pushed the master-hr-onboarding-feraz branch from 0d28e57 to 5ec108e Compare September 1, 2026 11:23
@feraz-odoo
feraz-odoo force-pushed the master-hr-onboarding-accural_ux-feraz branch from 3f01c80 to 7b0502c Compare September 1, 2026 11:40
@feraz-odoo
feraz-odoo force-pushed the master-hr-onboarding-feraz branch from 5ec108e to d6c97ce Compare September 1, 2026 11:53
@feraz-odoo
feraz-odoo force-pushed the master-hr-onboarding-accural_ux-feraz branch 3 times, most recently from b334ddb to 468da40 Compare September 3, 2026 07:38
Comment thread addons/hr_holidays/models/hr_leave_accrual_plan_level.py
@feraz-odoo
feraz-odoo force-pushed the master-hr-onboarding-accural_ux-feraz branch from 468da40 to 2fe78db Compare September 4, 2026 15:02
@feraz-odoo
feraz-odoo force-pushed the master-hr-onboarding-accural_ux-feraz branch from 2fe78db to 98193cb Compare September 7, 2026 14:22
to max day of the selected month
task-6522773
Closes#5259
@feraz-odoo
feraz-odoo force-pushed the master-hr-onboarding-accural_ux-feraz branch from 98193cb to 07e00da Compare September 7, 2026 14:27
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