Skip to content

[18.0][ADD] hr_holidays_allocation_domain - #233

Open
etobella wants to merge 1 commit into
OCA:18.0from
dixmit:18.0-wizard-domain
Open

[18.0][ADD] hr_holidays_allocation_domain#233
etobella wants to merge 1 commit into
OCA:18.0from
dixmit:18.0-wizard-domain

Conversation

@etobella

Copy link
Copy Markdown
Member

With this module, we are able to filter using complex domains on the allocation generation.

image

@etobella etobella changed the title [ADD] hr_allocation_generate_domain [18.0][ADD] hr_allocation_generate_domain Dec 22, 2025
@pedrobaeza

Copy link
Copy Markdown
Member

The module name should be prefixed by hr_holidays, so hr_holidays_allocation_domain may be a good candidate.

@etobella etobella changed the title [18.0][ADD] hr_allocation_generate_domain [18.0][ADD] hr_holidays_allocation_domain Dec 22, 2025
@etobella

Copy link
Copy Markdown
Member Author

@pedrobaeza comments attended 😉

@pedrobaeza pedrobaeza added this to the 18.0 milestone Dec 22, 2025
@github-actions

Copy link
Copy Markdown

There hasn't been any activity on this pull request in the past 4 months, so it has been marked as stale and it will be closed automatically if no further activity occurs in the next 30 days.
If you want this PR to never become stale, please ask a PSC member to apply the "no stale" label.

@github-actions github-actions Bot added the stale PR/Issue without recent activity, it'll be soon closed automatically. label Apr 26, 2026

@luisDIXMIT luisDIXMIT left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Code review and tested locally, LGTM!

@github-actions github-actions Bot removed the stale PR/Issue without recent activity, it'll be soon closed automatically. label May 3, 2026

@BhaveshHeliconia BhaveshHeliconia left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM!

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

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants