OCA / l10n-spain

Odoo Spain Localization
https://www.aeodoo.org/estado-localizacion
GNU Affero General Public License v3.0
292 stars 520 forks source link

[16.0][FIX] l10n_es_pos_sii: Set own channel for POS order SII jobs #3717

Closed aritzolea closed 2 months ago

aritzolea commented 2 months ago

Se crea un nuevo canal para los trabajos de envíos de los pedidos de TPV

aritzolea commented 2 months ago

@zamberjo podrías revisar esto? Gracias

pedrobaeza commented 2 months ago

¿Por qué un canal específico? Esto no es retro-compatible para las instancias existentes, que dejarían de tener la configuración correspondiente.

aritzolea commented 2 months ago

¿Por qué un canal específico? Esto no es retro-compatible para las instancias existentes, que dejarían de tener la configuración correspondiente.

Se ha especificado un canal específico porque los trabajos se estaban creando en el canal root. Por otra parte, hemos pensado no utilizar el canal root.invoice_validate_sii que se utiliza para las facturas para poder controlar el flujo de forma independiente.

pedrobaeza commented 2 months ago

OK, entendido entonces. En ese caso, entiendo que sí habría retrocompatibilidad, y no me parece mal el cambio.

OCA-git-bot commented 2 months ago

This PR has the approved label and has been created more than 5 days ago. It should therefore be ready to merge by a maintainer (or a PSC member if the concerned addon has no declared maintainer). 🤖

aritzolea commented 2 months ago

@pedrobaeza Se puede mergear?

pedrobaeza commented 2 months ago

/ocabot merge minor

OCA-git-bot commented 2 months ago

On my way to merge this fine PR! Prepared branch 16.0-ocabot-merge-pr-3717-by-pedrobaeza-bump-minor, awaiting test results.

OCA-git-bot commented 2 months ago

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