-
Notifications
You must be signed in to change notification settings - Fork 5
[ADD] l10n_uy_ux: certificate ux #382
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: 19.0
Are you sure you want to change the base?
Conversation
zaoral
commented
Jan 6, 2026
- hide private key not used
- save filename when loading certificate
* hide private key not used * save filename when loading certificate
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull request overview
Este PR mejora la experiencia de usuario (UX) del manejo de certificados para la localización uruguaya. Los cambios incluyen ocultar el campo de clave privada (public_key_id) para certificados de Uruguay y preservar el nombre del archivo al cargar certificados.
- Agrega el campo
content_filenamepara almacenar el nombre del archivo del certificado - Oculta el campo
public_key_idcuando el certificado es de Uruguay - Incrementa la versión del módulo de 1.1.0 a 1.2.0
Reviewed changes
Copilot reviewed 3 out of 3 changed files in this pull request and generated 2 comments.
| File | Description |
|---|---|
| l10n_uy_ux/models/certificate_certificate.py | Agrega el campo content_filename para almacenar el nombre del archivo del certificado |
| l10n_uy_ux/views/certificate_certificate_view.xml | Hereda la vista del certificado para conectar el campo content_filename con el widget de archivo y ocultar public_key_id para certificados uruguayos |
| l10n_uy_ux/manifest.py | Incrementa la versión a 19.0.1.2.0 y registra el nuevo archivo de vista en la lista de datos |
| <?xml version="1.0" encoding="UTF-8"?> | ||
| <odoo> | ||
| <record id="certificate_certificate_view_form" model="ir.ui.view"> | ||
| <field name="name">certificate_certificate_view_form.inherit.l10n_cl_edi</field> |
Copilot
AI
Jan 6, 2026
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
El nombre de la vista hace referencia a "l10n_cl_edi" (Chile) cuando debería hacer referencia a "l10n_uy_ux" (Uruguay). Esto puede causar confusión al buscar o mantener el código. Se sugiere cambiar el nombre a algo como "certificate_certificate_view_form.inherit.l10n_uy_ux".
| <field name="name">certificate_certificate_view_form.inherit.l10n_cl_edi</field> | |
| <field name="name">certificate_certificate_view_form.inherit.l10n_uy_ux</field> |
| class Certificate(models.Model): | ||
| _inherit = "certificate.certificate" | ||
|
|
||
| content_filename = fields.Char() |
Copilot
AI
Jan 6, 2026
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
El campo content_filename carece de atributos descriptivos. Se recomienda agregar al menos el atributo string para mejorar la claridad y mantenibilidad del código. Por ejemplo: content_filename = fields.Char(string="Certificate Filename")
| content_filename = fields.Char() | |
| content_filename = fields.Char(string="Certificate Filename") |
