Skip to content

Conversation

@edescalona
Copy link
Contributor

@BinhexTeam

It allows you to fill in the first and second invoice fields using a button, according to the date defined by the user, in the SII invoice tab.

@edescalona edescalona force-pushed the 17.0-add-l10n_es_aeat_sii_invoice_summary_pos branch from 1dff66b to 7d95358 Compare May 9, 2025 16:01
@edescalona edescalona marked this pull request as ready for review May 9, 2025 16:04
@edescalona
Copy link
Contributor Author

Ready for review

Copy link
Contributor

@christian-ramos-tecnativa christian-ramos-tecnativa left a comment

Choose a reason for hiding this comment

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

Revisa mis comentarios y dime que te parece. Gracias

@edescalona edescalona force-pushed the 17.0-add-l10n_es_aeat_sii_invoice_summary_pos branch from 3902fad to d10c9c5 Compare May 14, 2025 15:54
@edescalona
Copy link
Contributor Author

Hola @Christian-RB listo para revisión, gracias.

@edescalona edescalona force-pushed the 17.0-add-l10n_es_aeat_sii_invoice_summary_pos branch 2 times, most recently from 9202d1d to 82d40e7 Compare May 27, 2025 12:46
@edescalona edescalona force-pushed the 17.0-add-l10n_es_aeat_sii_invoice_summary_pos branch from 82d40e7 to 0fa0d10 Compare July 4, 2025 22:58
@christian-ramos-tecnativa
Copy link
Contributor

Vas a tener que volver a empujar la rama igualmente a ver si el CI sigue fallando

…los campos del Resumen de Factura Simplificada SII.
…s agrupadas por los impuestos.

[IMP] l10n_es_aeat_sii_invoice_summary_pos: Valor por defecto para el campo Facturas simplificadas en la AEAT en los partner

[IMP] l10n_es_aeat_sii_invoice_summary_pos: Validar campos de primer y última factura cuando no sea draft

[IMP] l10n_es_aeat_sii_invoice_summary_pos: Valor por defecto para el campo Facturas simplificadas en la AEAT en el partner Simplified Invoice Partner
@edescalona edescalona force-pushed the 17.0-add-l10n_es_aeat_sii_invoice_summary_pos branch from 0fa0d10 to 5cd45ae Compare October 31, 2025 22:33
@edescalona
Copy link
Contributor Author

listo @christian-ramos-tecnativa

@pedrobaeza pedrobaeza added this to the 17.0 milestone Nov 1, 2025
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.

No termino de entender la utilidad de esto, ya que así se estaría generando según entiendo doble contabilidad: la de la sesión al cerrar, y luego la de la factura de resumen, ¿o hay algo que se me escapa?

@christian-ramos-tecnativa
Copy link
Contributor

Ese problema ya lo presenta el módulo principal https://github.com/OCA/l10n-spain/tree/17.0/l10n_es_aeat_sii_invoice_summary, este lo único que pretende es tener una forma rápida de crear esas facturas agrupadas utilizando las ventas del tpv

@pedrobaeza
Copy link
Member

No, porque llevan finalidades distintas. El otro módulo pretende cubrir el hueco cuando el TPV no está en Odoo (el típico paquete de hardware + software de TPV que te venden y que utilizas tal cual, o N razones para no utilizar el de Odoo), y entonces solo introduces la contabilidad vía esta factura resumen, pero insisto, si el TPV es en Odoo, entonces con esto sí que tendrías doble contabilidad.

@christian-ramos-tecnativa
Copy link
Contributor

Estás acertado con el tema de la contabilidad doble y que usando este módulo tendrás que cancelar esos asientos o compensarlos de alguna manera, pero hay empresas que presentan estas facturas agrupadas por mes y lo normal es que las sesiones de odoo sean diarias. Por lo que este módulo viene a resolver ese punto sin necesidad de crearlo manualmente.

@pedrobaeza
Copy link
Member

La solución para ese caso debe ser enganchar el pos.session al sii.mixin, y enviar ese resumen desde ahí, no este módulo. Lo siento, pero no veo conveniente fusionar un módulo que promueve unas prácticas incorrectas.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants