[17.0][MIG] l10n_es_pos: Migration to 17.0 #3548

peluko00 · 2024-04-23T06:18:28Z

Module migrated to version 17.0

cc https://github.com/APSL 154343
@miquelalzanillas @lbarry-apsl @javierobcn @mpascuall please review

[Issue] Migration to version 17.0 #3298

pedrobaeza · 2024-04-23T06:52:09Z

/ocabot migration l10n_es_pos

lbarry-apsl

Validated functionality in Runboat

peluko00 · 2024-05-09T12:04:17Z

This module is available on Odoo core

pedrobaeza · 2024-05-09T12:07:21Z

Es correcto que ahora hay un l10n_es_pos en Odoo core, pero mi compañero @chienandalu tiene que confirmar que cubre correctamente todas las casuísticas. Os decimos luego.

peluko00 · 2024-05-09T12:10:35Z

Es correcto que ahora hay un l10n_es_pos en Odoo core, pero mi compañero @chienandalu tiene que confirmar que cubre correctamente todas las casuísticas. Os decimos luego.

Perfecto Pedro, vamos comentando!

NestorAgut · 2024-05-16T07:53:45Z

Es correcto que ahora hay un l10n_es_pos en Odoo core, pero mi compañero @chienandalu tiene que confirmar que cubre correctamente todas las casuísticas. Os decimos luego.

Perfecto Pedro, vamos comentando!

Nosotros hemos testeado el módulo y no nos genera la secuencia en los tickets.

chienandalu

Varios comentarios:

Lo primero que hay que hacer es analizar el gap entre lo que propone Odoo, ya que usó de base este mismo desarrollo para luego divergir en algunas cosas de forma bastante significativa.

Después de eso, hay dos formas de enfocar esta situación:

Extendiendo sobre lo que propone Odoo para cubrir necesidades.
Proponiendo una alternativa completamente distinta.

Y en cualquier caso, el nombre del módulo debe cambiar, ya que no pueden solaparse. Ya sea l10n_es_pos_oca o l10n_es_pos_extension.

Yo particularmente exploraría la posibilidad de acoplarse al estándar porque siempre es deseable reducir en lo posible la base de código a mantener por OCA. Esto, claro está, es más que probable que requiriese además scripts de migración.

Y si el gap resulta ser grande (por ejemplo, el requisito del offline es considerable) exploraría las vías de la extensión.

pedrobaeza · 2024-05-16T08:32:27Z

David, según parece, en el estándar se crea una factura por cada ticket, lo que lo hace totalmente inviable por la cantidad de apuntes contables que generaría, volviendo a la situación pre-14.

chienandalu · 2024-05-16T08:36:21Z

Sí, ese fue mi principal punto de crítica en su día (odoo/odoo#126268 (comment)).

pedrobaeza · 2024-05-16T08:54:47Z

Ese punto lo hace deshecharlo totalmente en mi opinión.

chienandalu · 2024-05-16T09:01:16Z

Ya, a mí tampoco me entusiasma 😅 pero entiendo que el enfoque puede ser válido si el número de transacciones no es desorbitado.

He vuelto a darle un repaso al código por encima y no se acopla muy bien con la alternativa OCA. De modo que la opción l10n_es_pos_oca sería la que tendría más peso.

miquelalzanillas · 2024-09-23T08:20:10Z

Hola @pedrobaeza ,

para ir avanzado en este módulo, proponemos los siguientes cambios:

Renombrar el módulo (l10n_es_pos_oca) que a su vez dependerá del módulo del core (l10n_es_pos)
Utilizar al máximo la funcionalidad nativa.
Saltar la creación de un asiento contable para cada factura simplificada y contabilitzar los tickets en el cierre de la sesión tal y como se hace en v16.

Si no nos dejamos nada, entendemos que con estos cambios recuperamos lo que teníamos en versiones anteriores.

Los ves bien?

chienandalu · 2024-09-23T08:45:31Z

Hola, @miquelalzanillas . Si crees que merece la pena acoplarlo al módulo nativo y no es muy costoso, adelante. Si no, tampoco pasa nada si siguen independientes, lo cuál probablemente sea menos trabajo

pedrobaeza · 2024-09-23T08:49:13Z

Mejor no depender del módulo original. Para qué tener dos instalados si uno anula el otro.

miquelalzanillas · 2024-09-23T08:52:09Z

Ok, entonces seria suficiente con renombrar el módulo.

Nos ponemos a ello en este mismo PR.

pedrobaeza · 2024-09-23T09:04:10Z

Para renombrar, utilizad estos comandos para mantener todo el historial en futuras versiones:

git filter-branch --tree-filter 'if [ -d l10n_es_pos ]; then mv l10n_es_pos l10n_es_pos_oca; fi' HEAD
git rebase origin/17.0

peluko00 · 2024-09-23T11:26:24Z

@miquelalzanillas @pedrobaeza @chienandalu Can you review please.

l10n_es_aeat_mod347/README.rst

chienandalu · 2024-09-24T09:58:10Z

l10n_es_pos_oca/static/src/js/Screens/TicketScreen/TicketScreen.js

+odoo.define("l10n_es_pos.TicketScreen", function (require) {
+    "use strict";
+
+    const TicketScreen = require("point_of_sale.TicketScreen");
+    const Registries = require("point_of_sale.Registries");
+
+    const L10nEsPosTicketScreen = (TicketScreen) =>
+        class extends TicketScreen {


No bloqueante, pero creo que sería recomendable adaptar el código js a ESM y usar patch para la extensión de modo que sigas las pautas actuales de Odoo core. Por ejemplo: https://github.com/OCA/OCB/blob/17.0/addons/l10n_ar_pos/static/src/overrides/components/ticket_screen/ticket_screen.js

Hecho, gracias por las anotaciones. Puedes revisarlo de nuevo cuando puedas

l10n_es_pos_oca/__manifest__.py

FernandoRomera · 2024-09-25T10:22:35Z

l10n_es_pos_oca/static/src/app/screens/receipt_screen/receipt/order_receipt.xml

+    >
+        <xpath expr="//t[@t-if='receipt.company.contact_address']" position="before">
+            <t t-if="env.pos.config.is_simplified_config and !receipt.to_invoice">
+                <div><t t-esc='receipt.date.localestring' /></div>


t-esc directives are also deprecated, but they still works as an alias of t-out. When possible, switch to this new name in advance of the future directive removal.

En plantillas estáticas hay matices: odoo/odoo#168191 (comment)

l10n_es_pos_oca/static/src/app/screens/payment_screen/payment_screen.js

chienandalu · 2024-09-25T07:17:30Z

l10n_es_pos_oca/static/src/app/screens/store/models.js

+   License AGPL-3.0 or later (https://www.gnu.org/licenses/agpl).
+*/
+
+/** @odoo-module */


Lo mismo con este

Sabes porque es debido este error:
9:1 error Parsing error: 'import' and 'export' may appear only with 'sourceType: module'

Sí, los modulos js deben estar nombrados con la extensión .esm.js para que el linter los repase adecuadamente

Hecho, gracias!

miquelalzanillas · 2024-09-26T06:52:08Z

Hola @peluko00 ,

He probado el módulo con los últimos cambios, pero sucede algo extraño con las secuencias.

Primero he abierto una sesión del POS y he creado un ticket de venta y otro de reembolso. Ha utilizado una secuencia especifica para el ticket de reeembolso. Yo quería usar la misma secuencia para ventas y reembolsos, así que he desactivado la opción "Usar secuencia específica para notas de crédito" en el diario de facturas simplicadas.

Después, he cerrado la sesión anterior, y he abierto una segunda sesión de POS. He vuelto a crear un ticket de venta y otro de reembolso, pero la secuencia simplificada de ambos tickets ahora es incorrecta, ha utilizado la secuencia de reembolso para ambos.

peluko00 · 2024-09-26T12:33:09Z

Hola @peluko00 ,

He probado el módulo con los últimos cambios, pero sucede algo extraño con las secuencias.

Primero he abierto una sesión del POS y he creado un ticket de venta y otro de reembolso. Ha utilizado una secuencia especifica para el ticket de reeembolso. Yo quería usar la misma secuencia para ventas y reembolsos, así que he desactivado la opción "Usar secuencia específica para notas de crédito" en el diario de facturas simplicadas.

Después, he cerrado la sesión anterior, y he abierto una segunda sesión de POS. He vuelto a crear un ticket de venta y otro de reembolso, pero la secuencia simplificada de ambos tickets ahora es incorrecta, ha utilizado la secuencia de reembolso para ambos.

Gracias por la anotación @miquelalzanillas, lo he resuelto. Lo podras revisar otra vez cuando puedas, gracias!

l10n_es_pos_oca/__manifest__.py

chienandalu

@peluko00 Convendría que probases mínimamente los cambios antes de pedir revisiones. Ahora mismo ni siquiera se cargan los assets correctamente...

chienandalu · 2024-09-30T08:21:38Z

l10n_es_pos_oca/__manifest__.py

+    "depends": ["point_of_sale"],
+    "data": ["views/pos_views.xml", "views/res_config_settings_views.xml"],
+    "assets": {
+        "point_of_sale.assets": [


Esto simplemente no puede funcionar, porque estos assets ya no existen: https://github.com/odoo/odoo/pull/120070/files

chienandalu · 2024-09-30T08:21:47Z

l10n_es_pos_oca/__manifest__.py

+            "l10n_es_pos_oca/static/src/app/screens/ticket_screen/ticket_screen.xml",
+            "l10n_es_pos_oca/static/src/app/screens/payment_screen/payment_screen.esm.js",
+            "l10n_es_pos_oca/static/src/app/store/models.esm.js",
+            "l10n_es_pos_oca/static/src/app/screens/ticket_screen/ticket_screen.js",


Esta ruta está mal

peluko00 · 2024-09-30T15:35:39Z

@peluko00 Convendría que probases mínimamente los cambios antes de pedir revisiones. Ahora mismo ni siquiera se cargan los assets correctamente...

Perdona @chienandalu se me olvido, he hecho una refactorización porque ha cambiado bastante en relación a la v16. Lo he probado y creo que funciona como deberia, le podrias hechar un ojo cuando puedas. Puede ser que algunas cosas se puedan mejorar. Gracias de antemano!

peluko00 mentioned this pull request Apr 23, 2024

Migration to version 17.0 #3298

Open

46 tasks

OCA-git-bot added this to the 17.0 milestone Apr 23, 2024

lbarry-apsl approved these changes Apr 26, 2024

View reviewed changes

peluko00 closed this May 9, 2024

peluko00 reopened this May 9, 2024

chienandalu reviewed May 16, 2024

View reviewed changes

pedrobaeza force-pushed the 17.0 branch from 7ca1be8 to dba08e9 Compare June 27, 2024 07:06

peluko00 force-pushed the 17.0-mig-l10n_es_pos branch from 019cde4 to 93fd52c Compare September 23, 2024 07:05

Endika Iglesias and others added 8 commits September 23, 2024 11:55

[ADD] Add l10n_es_pos

67b886a

[UPD] prefix versions with 8.0

4b8ff40

[FIX] remove en.po that was erroneously created by transbot

aa54c4f

[MIG] l10n_es_pos

d53d414

[ADD] l10n_es_pos: posbox support and translations

ced17ad

[MIG] l10n_es_pos: Migration to 11.0

590cad7

[FIX] l10n_es_pos: typo in oldname (OCA#886)

70a64ec

[IMP] l10n_es_pos: Handle the sequence properly (OCA#902)

0847a0e

pedrobaeza requested a review from chienandalu September 23, 2024 13:48

chienandalu suggested changes Sep 24, 2024

View reviewed changes

[IMP] l10n_es_pos_oca: pre-commit auto fixes

6710ff5

peluko00 force-pushed the 17.0-mig-l10n_es_pos branch 2 times, most recently from 1be5de5 to 81b6ea3 Compare September 25, 2024 07:13

FernandoRomera reviewed Sep 25, 2024

View reviewed changes

chienandalu suggested changes Sep 25, 2024

View reviewed changes

peluko00 force-pushed the 17.0-mig-l10n_es_pos branch 5 times, most recently from 5250ec2 to 4b70f1d Compare September 26, 2024 06:49

peluko00 force-pushed the 17.0-mig-l10n_es_pos branch 7 times, most recently from c16f7c7 to e756a54 Compare September 26, 2024 11:13

chienandalu reviewed Sep 27, 2024

View reviewed changes

l10n_es_pos_oca/__manifest__.py Outdated Show resolved Hide resolved

peluko00 force-pushed the 17.0-mig-l10n_es_pos branch from e756a54 to cdd4a75 Compare September 30, 2024 06:08

chienandalu suggested changes Sep 30, 2024

View reviewed changes

peluko00 force-pushed the 17.0-mig-l10n_es_pos branch 2 times, most recently from d0d4432 to c0b04e6 Compare September 30, 2024 15:31

[MIG] l10n_es_pos_oca: Migration to 17.0

941e971

peluko00 force-pushed the 17.0-mig-l10n_es_pos branch from c0b04e6 to 941e971 Compare October 1, 2024 06:26

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

[17.0][MIG] l10n_es_pos: Migration to 17.0 #3548

[17.0][MIG] l10n_es_pos: Migration to 17.0 #3548

peluko00 commented Apr 23, 2024 •

edited

Loading

pedrobaeza commented Apr 23, 2024

lbarry-apsl left a comment

peluko00 commented May 9, 2024

pedrobaeza commented May 9, 2024

peluko00 commented May 9, 2024

NestorAgut commented May 16, 2024 •

edited

Loading

chienandalu left a comment

pedrobaeza commented May 16, 2024

chienandalu commented May 16, 2024

pedrobaeza commented May 16, 2024

chienandalu commented May 16, 2024

miquelalzanillas commented Sep 23, 2024 •

edited

Loading

chienandalu commented Sep 23, 2024

pedrobaeza commented Sep 23, 2024

miquelalzanillas commented Sep 23, 2024

pedrobaeza commented Sep 23, 2024

peluko00 commented Sep 23, 2024

chienandalu Sep 24, 2024

peluko00 Sep 25, 2024

FernandoRomera Sep 25, 2024

chienandalu Sep 25, 2024

chienandalu Sep 25, 2024

peluko00 Sep 25, 2024

chienandalu Sep 25, 2024

peluko00 Sep 26, 2024

miquelalzanillas commented Sep 26, 2024

peluko00 commented Sep 26, 2024 •

edited

Loading

chienandalu left a comment

chienandalu Sep 30, 2024

chienandalu Sep 30, 2024

peluko00 commented Sep 30, 2024

[17.0][MIG] l10n_es_pos: Migration to 17.0 #3548

Are you sure you want to change the base?

[17.0][MIG] l10n_es_pos: Migration to 17.0 #3548

Conversation

peluko00 commented Apr 23, 2024 • edited Loading

pedrobaeza commented Apr 23, 2024

lbarry-apsl left a comment

Choose a reason for hiding this comment

peluko00 commented May 9, 2024

pedrobaeza commented May 9, 2024

peluko00 commented May 9, 2024

NestorAgut commented May 16, 2024 • edited Loading

chienandalu left a comment

Choose a reason for hiding this comment

pedrobaeza commented May 16, 2024

chienandalu commented May 16, 2024

pedrobaeza commented May 16, 2024

chienandalu commented May 16, 2024

miquelalzanillas commented Sep 23, 2024 • edited Loading

chienandalu commented Sep 23, 2024

pedrobaeza commented Sep 23, 2024

miquelalzanillas commented Sep 23, 2024

pedrobaeza commented Sep 23, 2024

peluko00 commented Sep 23, 2024

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

miquelalzanillas commented Sep 26, 2024

peluko00 commented Sep 26, 2024 • edited Loading

chienandalu left a comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

peluko00 commented Sep 30, 2024

peluko00 commented Apr 23, 2024 •

edited

Loading

NestorAgut commented May 16, 2024 •

edited

Loading

miquelalzanillas commented Sep 23, 2024 •

edited

Loading

peluko00 commented Sep 26, 2024 •

edited

Loading