Skip to content

[17.0][MIG] l10n_es_pos_sii: Migration to 17.0 #4450

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.

Sign up for GitHub

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

Jump to bottom
Open
wants to merge 28 commits into
base: 17.0
Choose a base branch
from
Open

[17.0][MIG] l10n_es_pos_sii: Migration to 17.0 #4450

wants to merge 28 commits into from

Conversation

@BernatObrador
Copy link
Contributor

@BernatObrador BernatObrador commented Oct 14, 2025 •
edited
Loading

Migración del módulo l10n_es_pos_sii

  • Se ha adaptado el código para usar las acciones planificadas en lugar de queue_job.
  • Se ha eliminado la dependencia de pos_default_partner, ahora simplemente usa el simplified_partner de l10n.

cc https://github.com/APSL 22768

@miquelalzanillas @lbarry-apsl @mpascuall @peluko00 @javierobcn @ppyczko please review

@BernatObrador BernatObrador force-pushed the 17.0-mig-l10n_es_pos_sii branch 2 times, most recently from 8035f55 to b35c6a8 Compare October 14, 2025 05:55
@pedrobaeza
Copy link
Member

Hola, el nombre debería ser l10n_es_pos_sii_oca en todo caso, ¿pero por qué cambiarlo? Otra cosa que tampoco me convence mucho es depender de pos_default_partner. Creo que eso habría que manejarlo de otra forma.

@BernatObrador
Copy link
Contributor Author

@pedrobaeza Lo cambié dado que se cambió el nombre del módulo de la OCA de l10n_es_pos a l10n_es_pos_oca, dado que en el core apareció el l10n_es_pos, simplemente para seguir con la nomenclatura, pero no pasa nada, puedo volver a dejarlo como l10n_es_pos_sii.

Lo de pos_default_partner puedo revisar para no depender de él, claro, me parece buena opción.

Gracias por tu tiempo.

@pedrobaeza
Copy link
Member

Ya, el de l10n_es_pos se cambió para no colisionar con el de Odoo. Odoo por ahora no tiene SII para el PoS, y no sé si algún día lo añadirá. Podemos renombrarlo preventivamente, o dejarlo así hasta que colisione.

Jose Zambudio and others added 25 commits October 14, 2025 09:19
@BernatObrador
[REF] account.move extraemos parte de sii a un mixin.
6bb18c8 
[ADD] módulo l10n_es_pos_sii para permitir enviar pedidos del TPV al sii

Roadmap:
 - Comprobar que funcionalidad de account.move sigue siendo correcta.
 - Posibilidad de enviar al sii una sesión (tipo summary)
 - Envío devoluciones al SII -> POS
 - Cancelar envío al SII -> POS
@almumu @BernatObrador
[15.0][FIX] l10n_es_pos_sii: default partner y roadmap.
c9c9eaa 
OCA#3088

Los pedidos se enviarán siempre como F2, sin especificar la contraparte. Aún así, en el _sii_get_partner que también se utiliza en otros métodos del sii, hacemos que el partner que obtiene sea el default partner del pos, para no generar incongruencias.
Respecto a la fecha, de momento seguimos enviado la del pedido.
@ljsalvatierra-factorlibre @BernatObrador
[I18N] l10n_es_pos_sii: es.po
d36f3b5
@ljsalvatierra-factorlibre @BernatObrador
[FIX] l10n_es_pos_sii: PoS Order view SII tab
ff58f52
@ljsalvatierra-factorlibre @BernatObrador
[IMP] l10n_es_pos_sii: PoS Orders search view
c511a4f
@ljsalvatierra-factorlibre @BernatObrador
[FIX] l10n_es_pos_sii: sii_pos_description compute
4197d6a
@ljsalvatierra-factorlibre @BernatObrador
[FIX] l10n_es_pos_sii: sii_refund_type
4a2ddcb
@ljsalvatierra-factorlibre @BernatObrador
[IMP] l10n_es_pos_sii: PoS order view Queue Jobs
9bf805f
@ljsalvatierra-factorlibre @BernatObrador
[UPD] l10n_es_pos_sii: make addon autoinstallable
b646083
@BernatObrador
[UPD] Update l10n_es_pos_sii.pot
3350481
@OCA-git-bot @BernatObrador
[UPD] README.rst
16f9f45
@OCA-git-bot @BernatObrador
[ADD] icon.png
4718fac
@ljsalvatierra-factorlibre @BernatObrador
[FIX] l10n_es_pos_sii: don't allow invoicing order in a closed session
7edbfc9
@BernatObrador
[UPD] Update l10n_es_pos_sii.pot
0d2a056
@OCA-git-bot @BernatObrador
[BOT] post-merge updates
f3e4e1d
@aritzolea @BernatObrador
[FIX] l10n_es_pos_sii: Set auto_install correctly on manifest
0a997c2
@OCA-git-bot @BernatObrador
[BOT] post-merge updates
8b30c35
@Ivorra78 @BernatObrador
Translated using Weblate (Spanish)
4a94b53 
Currently translated at 100.0% (46 of 46 strings)

Translation: l10n-spain-16.0/l10n-spain-16.0-l10n_es_pos_sii
Translate-URL: https://translation.odoo-community.org/projects/l10n-spain-16-0/l10n-spain-16-0-l10n_es_pos_sii/es/
@BernatObrador
[IMP] Refactor para futura implementación Veri*FACTU.
d77cdd5 
Creación de un nuevo modelo en `l10n_es_aeat` llamado *aeat_mixin*, del
que heredan *sii.mixin* (ya existente) y un nuevo *verifactu.mixin*.
Refactor de *sii.mixin* pasando funcionalidad a *aeat_mixin*
Se han renombrado varios campos sii_* a aeat_* ya que serán utilizados
por ambos.
@BernatObrador
[UPD] Update l10n_es_pos_sii.pot
f5e9044
@OCA-git-bot @BernatObrador
[BOT] post-merge updates
601876f
@weblate @BernatObrador
Update translation files
d3ef17e 
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate.

Translation: l10n-spain-16.0/l10n-spain-16.0-l10n_es_pos_sii
Translate-URL: https://translation.odoo-community.org/projects/l10n-spain-16-0/l10n-spain-16-0-l10n_es_pos_sii/
@aritzolea @BernatObrador
[FIX] l10n_es_pos_sii: Set own channel for POS order SII jobs
cb153c7
@OCA-git-bot @BernatObrador
[BOT] post-merge updates
00cd507
@BernatObrador
[IMP]l10n_es_pos_sii: Added group to sii_registration_key and sii_reg…
a9b7fe1 
…istration_key_code to allow the pos users to interact whit the pos orders.
@BernatObrador BernatObrador force-pushed the 17.0-mig-l10n_es_pos_sii branch from b35c6a8 to 4ad891a Compare October 14, 2025 07:21
@BernatObrador BernatObrador force-pushed the 17.0-mig-l10n_es_pos_sii branch from 4ad891a to 9d34dd5 Compare October 14, 2025 07:40
@BernatObrador BernatObrador marked this pull request as ready for review October 14, 2025 07:50
@BernatObrador
Copy link
Contributor Author

@pedrobaeza Hecho, dime si ves alguna cosa más.

@pedrobaeza
Copy link
Member

/ocabot migration l10n_es_pos_sii

@OCA-git-bot OCA-git-bot added this to the 17.0 milestone Oct 14, 2025
@OCA-git-bot OCA-git-bot mentioned this pull request Oct 14, 2025
Open
63 tasks
pedrobaeza
pedrobaeza approved these changes Oct 14, 2025
View reviewed changes
Copy link
Member

@pedrobaeza pedrobaeza left a comment

Choose a reason for hiding this comment

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

Code review. Necesitaría, eso sí, que alguien pueda confirmar la parte JS.

@BernatObrador
Copy link
Contributor Author

Hola, @etobella.

¿Si tienes tiempo podrías revisar esta PR?
¡Muchas gracias de antemano!

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

Reviewers

@pedrobaeza pedrobaeza pedrobaeza approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

17.0

Development

Successfully merging this pull request may close these issues.

8 participants

@BernatObrador @pedrobaeza @OCA-git-bot @almumu @ljsalvatierra-factorlibre @aritzolea @Ivorra78 @weblate