search

OCA / l10n-spain

Odoo Spain Localization
https://www.aeodoo.org/estado-localizacion
GNU Affero General Public License v3.0
292 stars 520 forks source link

[16.0][FIX] l10n_es_*: Fix minimum models read permissions for readonly acc… #3550

Closed aritzolea closed 7 months ago

aritzolea commented 7 months ago

…ounting group

El usuario con el permiso Mostrar funciones de contabilidad: solo lectura no puede acceder a las facturas porque hay unos registros sobre los que no tiene acceso de lectura.

Se agregan los permisos que faltan, para que el usuario con estos permisos pueda acceder a las facturas sin problema.

OCA-git-bot commented 7 months ago

Hi @pedrobaeza, @ao-landoo, some modules you are maintaining are being modified, check this out!

pedrobaeza commented 7 months ago

Evita por favor poner la palabra WIP en los títulos. Con marcarlo como draft es suficiente. También es conveniente no renombrar XML-IDs. Por último, poner permisos explícitos al grupo mencionado no creo que sea la solución, si no proteger con groups el campo que impide la lectura.

aritzolea commented 7 months ago

Evita por favor poner la palabra WIP en los títulos. Con marcarlo como draft es suficiente. También es conveniente no renombrar XML-IDs. Por último, poner permisos explícitos al grupo mencionado no creo que sea la solución, si no proteger con groups el campo que impide la lectura.

Campos protegidos por grupo. También he comprobado en el runboat que ahora el usuario con el permiso Mostrar funciones de contabilidad: solo lectura puede leer las facturas.

aritzolea commented 7 months ago

@pedrobaeza Lo ves bien para mergear?

OCA-git-bot commented 7 months ago

What a great day to merge this nice PR. Let's do it! Prepared branch 16.0-ocabot-merge-pr-3550-by-pedrobaeza-bump-minor, awaiting test results.

OCA-git-bot commented 7 months ago

Congratulations, your PR was merged at 7f08c9b2a3f3bdf8233bce6c3c51448c14bd5f74. Thanks a lot for contributing to OCA. ❤️