Skip to content

[18.0][ADD] hr_collective_agreement_partner#1545

Draft
Anxo82 wants to merge 2 commits intoOCA:18.0from
sygel-technology:18.0-add-hr_collective_agreement_partner
Draft

[18.0][ADD] hr_collective_agreement_partner#1545
Anxo82 wants to merge 2 commits intoOCA:18.0from
sygel-technology:18.0-add-hr_collective_agreement_partner

Conversation

@Anxo82
Copy link

@Anxo82 Anxo82 commented Feb 5, 2026

@HaraldPanten @Jaimermaccione

@Anxo82 Anxo82 marked this pull request as draft February 5, 2026 07:37
@Anxo82 Anxo82 force-pushed the 18.0-add-hr_collective_agreement_partner branch from 543a4d8 to 0d6fe51 Compare February 5, 2026 09:00
@Anxo82 Anxo82 force-pushed the 18.0-add-hr_collective_agreement_partner branch from 0d6fe51 to dc2dfd7 Compare February 5, 2026 09:30
Copy link

@NL66278 NL66278 left a comment

Choose a reason for hiding this comment

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

👍 Thank you also for this module. Again a few changes would help adoption.

@Anxo82 Anxo82 force-pushed the 18.0-add-hr_collective_agreement_partner branch from dc2dfd7 to 7076383 Compare February 5, 2026 11:34
@Anxo82
Copy link
Author

Anxo82 commented Feb 5, 2026

@NL66278 the ir.actions.act_window has been moved to its own XML file, and inherited views now keep the original XML-IDs, with the view name updated to include the module name as suggested.
Thanks.

Copy link

@NL66278 NL66278 left a comment

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

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants