[17.0][IMP] l10n_es_igic: Add DUA #4097
Conversation
|
Que les parece @etobella @pedrobaeza @julianram? |
|
Al intentar probarlo en el runboat da un error pero está resuelto aqui #4099 |
christian-ramos-tecnativa
force-pushed
the
17.0-l10n_es_igic-add-dua
branch
from
9499dae to
8db3658
Compare
|
Rebase hecho para poder testear en el runboat |
|
Christian, podrías incluir un test post-install en este módulo que cree una compañía y cargue el plan contable para así detectar problemas como el anterior sobre la marcha. |
christian-ramos-tecnativa
force-pushed
the
17.0-l10n_es_igic-add-dua
branch
from
8db3658 to
c26de4f
Compare
|
@pedrobaeza Exactamente el mismo error 😞 |
christian-ramos-tecnativa
force-pushed
the
17.0-l10n_es_igic-add-dua
branch
from
c26de4f to
e05babc
Compare
christian-ramos-tecnativa
force-pushed
the
17.0-l10n_es_igic-add-dua
branch
2 times, most recently
from
a5c46b0 to
6393c58
Compare
|
Trabajando en los scripts de migración actualmente |
christian-ramos-tecnativa
force-pushed
the
17.0-l10n_es_igic-add-dua
branch
from
6393c58 to
107d091
Compare
|
Probado y listo para revisar |
l10n_es_igic/readme/DESCRIPTION.md
Outdated
| @@ -1,2 +1,3 @@ | |||
| Módulo que amplía la contabilidad española introduciendo los impuestos y | |||
| posiciones fiscales para el IGIC. | |||
| Para más información sobre las importaciones con DUA acceder al siguiente enlace [README](https://github.com/OCA/l10n-spain/blob/16.0/l10n_es_dua_igic/README.rst) | |||
There was a problem hiding this comment.
Esto es una referencia a una versión anterior. Se debería incluir y actualizar el texto directamente, ya que los nombres no son iguales ni la forma de proceder.
There was a problem hiding this comment.
Poner el texto en esta versión no es problema, pero, ¿A que diferencias te refieres? Cuando lo leí en su momento me pareció una explicación aplicable a esta versión ya que mantuve los mismos ids y nombres, ¿es el proceso distinto?
There was a problem hiding this comment.
La posición fiscal no debería ser la misma (la incluye el core), así como los impuestos, y ahora se hace con un grupo de impuestos, no con lo de la compensación.
There was a problem hiding this comment.
Gracias por la orientación, tanto la posición fiscal como los impuestos se han mantenido específicos para la ATC, voy a indagar un poco más por si lo podemos poner común, al igual que los IRPF en 18.0. Revisaré el proceso por grupos de impuestos para definirlo en el README
There was a problem hiding this comment.
Hola @pedrobaeza entonces entiendo que tenemos que reutilizar la posición fiscal del core "DUA" añadiendo los mapeos pertinentes hacía un nuevo grupo de impuestos según porcentaje, añadiendo el "Substract Base" para aplicar la compensación.
Por otro lado, se tendrá que modificar la documentación para dar claridad a su uso.
There was a problem hiding this comment.
Por "este lado" no sé a quién te refieres, jeje. Si acaso, entonces trae esos cambios a este PR.
There was a problem hiding this comment.
Claro, en lo que lo aprueben lo traigo a este PR. Gracias
There was a problem hiding this comment.
Ya se está revisando desde "este lado" jajaja, gracias por el seguimiento @pedrobaeza
christian-ramos-tecnativa
force-pushed
the
17.0-l10n_es_igic-add-dua
branch
from
3d5dbc8 to
db3e997
Compare
|
Hola, alguien para los fallos de l10n_es_bien_inversion de los tests? |
|
Podéis hacer rebase sobre la última rama 17.0, que tiene el fix para el cambio que hizo Odoo upstream. |
christian-ramos-tecnativa
force-pushed
the
17.0-l10n_es_igic-add-dua
branch
from
db3e997 to
38631ae
Compare
5 participants
Teniendo en cuenta lo complicado que es extender localizaciones desde módulos externos y siguiendo el ejemplo de l10n_es considero que lo mejor sería incluir los impuestos/productos DUA en el l10n_es_igic directamente
@BinhexTeam T13227