-
-
Notifications
You must be signed in to change notification settings - Fork 520
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
[16.0][MIG] l10n_es_facturae: Migration to 16.0 #2707
Conversation
Nombre del PR incorrecto. |
f6a9b6f
to
24e4acf
Compare
@pedrobaeza Ya me he dado cuenta, ya está modificado. |
b30b6dd
to
d85c6e5
Compare
/ocabot migration l10n_es_facturae |
d85c6e5
to
8b853d1
Compare
/ocabot rebase |
@etobella The rebase process failed, because command
|
@ramiadavid Can you rebase it? it is failing due to test-requirements |
- Se han renombrado los módulos para usar la nomenclatura propuesta por OpenERP: l10n_es para el módulo base de localización (plan de cuentas), l10n_es_* para el resto de módulos. - Se eliminan los módulos extra_addons/* que deberían moverse a los extra-addons genéricos (no son específicos de España). - Se renombran los __terp__.py por __openerp__.py
…ciertos módulos, por los correspondientes que se modificaron para esta versión 6.0 y en ciertos __init__ adaptamos los imports a los nuevos nombres de los ficheros. Renombrado de los archivos de traducción españoles de es_ES.po a es.po y pequeñas refactorizaciones
…ódulos, limpieza l10n_es_partner_mercantil
…ron marcados como no instalables mientras no se compruebe que funcionan o migren, para poder usarlos o probarlos se debería poner el atributo installable de los ficheros __openerp__.py de cada módulo a True
[FIX] l10n_es_lopd: Nombre de categoría con tilde, que da error al importar el repositorio entero.
…er eliminado los templates.
Exactamente en lo del código postal |
@extrememicro Rectifico, tienes razón, el xml no lo está generando correctamente, ya lo modifico y lo subo |
b1d7b0f
to
27a39f2
Compare
Gracias @ramiadavid . Lo del código postal lo dejamos para un fix posterior que me apunto para hacer en los Oca days. |
f5cf461
to
2eab71d
Compare
@extrememicro Creo que ya está correcto |
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.
@ramiadavid
Genera el xml firmado pero si seleccionas que anexe el pdf al mismo en res.partner da error:
File "/mnt/data/odoo-addons-dir/l10n_es_facturae/models/account_move.py", line 177, in _get_facturae_move_attachments
content, content_type = action._render(self.ids)
TypeError: IrActionsReport._render() missing 1 required positional argument: 'res_ids'
2eab71d
to
417c32e
Compare
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.
@ramiadavid
Faltarían estos pequeños cambios. Gracias
417c32e
to
bc208d0
Compare
@etobella LGTM! |
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.
LGTM
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.
LGTM Probado en runboat
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.
@etobella gracias. Ya le puedes hacer merge. Aprové los cambios.
This PR has the |
/ocabot merge nobump |
Hey, thanks for contributing! Proceeding to merge this for you. |
It looks like something changed on |
Congratulations, your PR was merged at e8ec74a. Thanks a lot for contributing to OCA. ❤️ |
#2526