Skip to content
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

[ADD][12.0] l10n_es_extra_data: new module #2420

Merged
merged 1 commit into from
Jul 21, 2022

Conversation

cubells
Copy link
Member

@cubells cubells commented Jul 19, 2022

Nuevo módulo para añadir impuestos que no se pueden añadir en la versión 12.0 de odoo.

@cubells cubells force-pushed the 12.0-l10n_es_extra_data branch from b171f34 to b92f6dc Compare July 20, 2022 08:22
@cubells cubells force-pushed the 12.0-l10n_es_extra_data branch from b92f6dc to 7a8dc25 Compare July 20, 2022 08:24
Copy link
Contributor

@HaraldPanten HaraldPanten left a comment

Choose a reason for hiding this comment

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

Está perfecto 👍

@pedrobaeza pedrobaeza added this to the 12.0 milestone Jul 20, 2022
@pedrobaeza
Copy link
Member

Sí, correcto. Ahora la siguiente decisión será si ampliar en el 303 las dependencias con este módulo o crear un módulo extra.

@cubells
Copy link
Member Author

cubells commented Jul 20, 2022

Jo creo que hay que ampliar las dependencias con este módulo. Mejor no?

@pedrobaeza
Copy link
Member

Ya, bueno, puede causar algo de fricción, pero aquí lo que queráis los que seguís usando 12.0.

@cubells
Copy link
Member Author

cubells commented Jul 20, 2022

O creamos otro módulo, a mi me da igual. Lo que mejor creas.

Podíamos aprobar este no?

@HaraldPanten
Copy link
Contributor

Yo ampliaría las dependencias del 303 (o cualquier modelo afectado: libro de IVA, sii...), pero una vez se vea afectado por estos cambios.

De momento no veo necesidad, ¿no?

@pedrobaeza
Copy link
Member

El 303 y el vat book sí que lo necesita.

@HaraldPanten
Copy link
Contributor

El 303 y el vat book sí que lo necesita.

Sí, por eso. Digo de no ampliar las dependencias aquí (l10n_es_extra_data).

La idea sería añadir la dependencia en el 303, en el vat_book, etc. pero no hacerlo hasta que no se haga la correspondiente ampliación. Como se hace siempre, vaya...

No sé si me explico.

@pedrobaeza
Copy link
Member

Bueno, desde el principio lo que planteábamos era ampliar dependencias de l10n_es_aeat_mod303 o crear un l10n_es_aeat_mod303_extra_data, no lo que tú planteas.

@HaraldPanten
Copy link
Contributor

Bueno, desde el principio lo que planteábamos era ampliar dependencias de l10n_es_aeat_mod303 o crear un l10n_es_aeat_mod303_extra_data, no lo que tú planteas.

Igual no os he entendido bien, disculpa. Pues me convence más la opción del l10n_es_aeat_mod303_extra_data para tenerlo bien segmentado.

@cubells
Copy link
Member Author

cubells commented Jul 21, 2022

Hago yo ese módulo?

@pedrobaeza
Copy link
Member

Adelante por mi parte. Nosostros ya no trabajamos con v12.

@pedrobaeza
Copy link
Member

/ocabot merge nobump

@OCA-git-bot
Copy link
Contributor

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

@HaraldPanten
Copy link
Contributor

Hago yo ese módulo?

Por mi parte también, si tienes disponibilidad.

Gracias.

@OCA-git-bot OCA-git-bot merged commit 240257a into OCA:12.0 Jul 21, 2022
@OCA-git-bot
Copy link
Contributor

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants