[ADD] [10.0] l10n_es_aeat_sii_invoice_summary

[omar7r]

Se adapta el módulo l10n_es_aeat_sii_invoice_summary a la 10.0 desde el PR #796

[pedrobaeza]

@omar7r puedes mirar por qué esto está rojo? Quizás hacer un rebase?

[omar7r]

Quedan arreglados los problemas con el flake8

[pedrobaeza]

@omar7r esto sigue teniendo errores de Travis

[omar7r]

@pedrobaeza mira el detalle, está todo bien excepto el despliegue con ocb que es como si se parara, en despliegue con odoo si se hizo

[pedrobaeza]

OK, reviso.

[rafaelbn]

Hola @omar7r , esto entiendo que lo tenéis en producción hace tiempo. ¿Podrías por favor revisar travis 🔴 para que esté listo para mergear? Gracias!

[pedrobaeza]

Quedaba la decisión de si ponerlo en el módulo principal o no, ya que la dependencia del POS no era técnica.

[omar7r]

Hola, @rafaelbn @pedrobaeza sí ya hace tiempo que lo tenemos en producción, tanto en la 10.0 como en la 11.0, pero no lo veo como para incluir en el base del SII, está bien en casos concretos, tpvs externos por ejemplo, con este módulo, si usamos el TPV de Odoo estamos apuntando doblemente el ingreso y los impuestos y hay que regularizar contablemente una vez enviado el resumen al SII o revertirlo, por lo que no deja de ser un workaround que ya tenía hecho Factorlibre en la 11, pero no para incluir en el módulo base en mi opinión, la forma más compatible con Odoo, si usas el TPV de Odoo sería enviar cada una de las facturas simplificadas individualmente, auque tampoco me encanta por el número de interacciones, de esto me suena que tiene algo Acysos en sus módulos del SII para enviar pos.orders, otra posibilidad sería enviar el resumen desde los TPV.
Por esto, yo lo mantendría así, como un módulo aparte para los casos concretos o como posibilidad para gestionar las facturas simplificadas del TPV de Odoo, pero conociendo sus limitaciones al implantarlo.

Voy a eliminar la dependencia de point_of_sale para ver si al volver a lanzarse ya no falla Travis, el error actual no es de este módulo

[pedrobaeza]

Has hecho rebase? Toda esta explicación tan buena que has puesto, sería conveniente ponerla en el propio README para ser conscientes del workaround como bien dices.

[pedrobaeza]

He hecho squash de los commits de Omar y generado el README por fragmentos. En cuanto dé verde, fusiono.

Gracias por la paciencia.

[pedrobaeza]

El CI está rojo por un error que ha aflorado en otro módulo. Estamos mirando a ver por qué puede ser.

[pedrobaeza]

Bueno, mira, como la rama principal está en rojo de todas formas, fusiono el módulo, y cuando veamos el error, se resuelve en ella, pero no dejamos más tiempo esto.

Gracias por el PR.