[18.0][ADD] l10n_es_hr_collective_agreement - #4855
Conversation
|
@Anxo82 Puedes hacer el README en castellano, por favor? En la localización podemos gestionarlo así. Gracias. |
cd283f5 to
abbd914
Compare
|
@HaraldPanten Ya he puesto en castellano el README. |
Jaimermaccione
left a comment
Jaimermaccione
left a comment
There was a problem hiding this comment.
@Anxo82 The module has been functionally tested in a local environment.
Installation is straightforward and does not require any additional configuration.
Once installed, it correctly creates the predefined Spanish master data needed for the Collective Agreement module, including:
- Official publications (e.g. BOE, DOGC)
- Agreement scopes (Autonomous, Provincial, etc.)
It behaves as expected and properly supports the Spanish regulatory setup for collective agreements.
LGTM 👍
Note (non-blocking)
If you already have records created with the same name in Agreement Scope or Official Publications, the installation will fail due to the unique constraint on the name field.
This is technically correct from a data integrity perspective, but it’s something to keep in mind.
|
@HaraldPanten Could you please review it ? |
HaraldPanten
left a comment
HaraldPanten
left a comment
There was a problem hiding this comment.
/ocabot merge nobump
|
On my way to merge this fine PR! |
|
Congratulations, your PR was merged at ed93530. Thanks a lot for contributing to OCA. ❤️ |
5 participants
@HaraldPanten @Jaimermaccione
Adds a complementary module providing initial Spanish configuration data for collective agreements. Requires hr_collective_agreement.