Skip to content

[18.0] [IMP] l10n_es_vat_book: Add igic data - #4506

Closed
luisDIXMIT wants to merge 1 commit into
OCA:18.0from
dixmit:18.0-imp-l10n_es_vat_book
Closed

[18.0] [IMP] l10n_es_vat_book: Add igic data#4506
luisDIXMIT wants to merge 1 commit into
OCA:18.0from
dixmit:18.0-imp-l10n_es_vat_book

Conversation

@luisDIXMIT

Copy link
Copy Markdown
Contributor

Como en la versión 18.0 se ha integrado la localización canaria dentro del módulo de la localización española, creo que es conveniente integrar la información que contiene el módulo l10n_es_vat_book_igic dentro de este, en vez de migrar el módulo.

id,name
common_472,account_common_472
common_477,account_common_477
common_canary_4727,account_common_canary_4727

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Aquí y en el mapeo tienes que cambiar a account_common_canary_4777

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Hola Pedro, gracias por tu respuesta. Ya he hecho el cambio de cuenta pero quisiera saber por qué no debo mantener la cuenta account_common_canary_4727 (HP. IVA soportado)

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Faltaría esto, y la razón Luis es que se ha cambiado la lógica para que ahora se excluyan cuentas en lugar de incluirlas, y por eso hay que excluir la 4777 en lugar de incluir la 4727. Enric de todas formas está haciendo algunos cambios según me ha dicho.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Ah, vale gracias!! Sí está trabajando en ello.

@etobella
etobella force-pushed the 18.0-imp-l10n_es_vat_book branch from 9f5f43b to 1373ff4 Compare November 9, 2025 21:49
@luisDIXMIT
luisDIXMIT force-pushed the 18.0-imp-l10n_es_vat_book branch 2 times, most recently from e9b75d3 to 8b6045f Compare November 10, 2025 07:34
@luisDIXMIT

Copy link
Copy Markdown
Contributor Author

He hecho rollback de mi cambio porque he matado el que había hecho @etobella sin querer, mantengo su propuesta.

@etobella
etobella force-pushed the 18.0-imp-l10n_es_vat_book branch from 8b6045f to ab2dbd0 Compare January 19, 2026 17:47
@luisDIXMIT
luisDIXMIT force-pushed the 18.0-imp-l10n_es_vat_book branch from ab2dbd0 to f41f0df Compare February 24, 2026 14:35
@luisDIXMIT
luisDIXMIT force-pushed the 18.0-imp-l10n_es_vat_book branch from f41f0df to a884294 Compare February 24, 2026 14:39
@github-actions

Copy link
Copy Markdown

There hasn't been any activity on this pull request in the past 4 months, so it has been marked as stale and it will be closed automatically if no further activity occurs in the next 30 days.
If you want this PR to never become stale, please ask a PSC member to apply the "no stale" label.

@github-actions github-actions Bot added the stale PR/Issue without recent activity, it'll be soon closed automatically. label Jun 28, 2026
syci added a commit to syci/l10n-spain that referenced this pull request Jul 6, 2026
syci added a commit to syci/l10n-spain that referenced this pull request Jul 6, 2026
Complemento del mapa Canarias de OCA OCA#4506: incluir igic_sop_*_s en
IGIC Soportado y test con compra IGIC_SOP_7_S (caso real 22 GRADOS).
@github-actions github-actions Bot closed this Aug 2, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

stale PR/Issue without recent activity, it'll be soon closed automatically.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants