[14.0][IMP] l10n_es_pos: Secuencias por dispositivo #2812
Conversation
In 2967122 the behavior of the simplified sequence was change to gather via rpc for each ticket validation. For the case of offline the planned fallback was to set the unique ID given by Odoo. This change fallbacks to the former behavior for that case allowing to obtain the proper sequence even offline. The main drawback will be that concurrent users in the same session could collide their sequence numbers if they happen to be offline. Anyway, that would be easily fixed just by separating such cashiers in separate configs. As a solution to that, a different technique should be used (longpoll for multi sessions or something like that)
ao-landoo
force-pushed
the
14.0-imp-l10n_es_pos-seq_by_device
branch
3 times, most recently
from
cd92dfb to
ef89a7d
Compare
ao-landoo
force-pushed
the
14.0-imp-l10n_es_pos-seq_by_device
branch
from
ef89a7d to
afd9591
Compare
|
Listo para review @pedrobaeza @anajuaristi @chienandalu @omar7r @AntoniRomera @blaskurain @Mat-moran |
ao-landoo
force-pushed
the
14.0-imp-l10n_es_pos-seq_by_device
branch
from
afd9591 to
d89502c
Compare
|
Muchas gracias por el PR 😄 el runbot da el siguiente error No he podido probar en profundidad, lo he instalado en local y veo que te deja seleccionar un dispositivo en un punto de venta aunque no lo tenga asignado en la configuración. Para replicar
Seguiré testeando. Otra cosilla. |
Respecto al error del runboat, parece que es un problema funcional que no está relacionado con este PR. Cuando un punto de venta no tiene seleccionado ningún dispositivo permite seleccionar todos los de esa compañía mientras no estén bloqueados. Lo pone en la 'ayuda' del campo: Si se deja vacío, se pueden seleccionar todos los dispositivos. ¿Te refieres a eso? En ese caso es el comportamiento esperado. El commit del PR #2582 está incluido en este PR. |
ao-landoo
force-pushed
the
14.0-imp-l10n_es_pos-seq_by_device
branch
from
d89502c to
e654361
Compare
En ese caso el funcionamiento es correcto. |
|
Hola, @omar7r @chienandalu hay consenso? Lo digo porque ya está el PR a v16 y deberíamos entra en v15 con consenso y coherencia, de hecho ahora mismo (no tengo pos) no sé cómo está en v15 😄 Mil gracias a todos por los esfuerzos de explicar, compartir y debatir la mejor solución en todos los escenarios. Saludos |
There was a problem hiding this comment.
Muchas gracias por el esfuerzo @ao-landoo y @gelo-landoo . En cualquier caso, todo el tema de los pos.device debería ir en un módulo general a OCA/pos y, o bien hacer depender a l10n_es_pos de aquel o bien hacer un tercer módulo que unifique ambos para esa necesidad (abriendo hooks los hooks fuesen necesarios en l10n_es_pos).
|
Lo veo perfecto, al comentario de @chienandalu no creo que sea necesario separarlo, seria triplicar trabajo, ya que se tiene que modificar mucho el modulo base l10_es_pos y el otro quedaria con 3 lineas de codigo |
|
En mi opinión llevarlo a la repo OCA/pos actualmente no tiene sentido porque el concepto de dispositivos POS es super simple y no tienen funcionalidad propia ahora mismo. Si en un futuro se piensa agregar nuevas funcionalidades si que me parece adecuado llevar la definición de este modelo a ese repositorio, pero ahora mismo quedaría un módulo muy simple sin ninguna utilidad. |
|
Pues todo esto no debería estar en el módulo principal. Debería ser un módulo |
|
No coincido en que esta funcionalidad debería de ir en otro módulo separado. El módulo principal ( Ahora mismo el módulo no lo adapta a todos los casos de uso reales porque en caso de utilizar mas de un dispositivo en una sesión hay casos en los que se imprime el mismo número de factura simplificada, cosa que no está permitido por la legislación española. Agregar esto al módulo Aparte de que no considero que se trate de una funcionalidad que haga que el código del módulo sea mucho mas complejo. Simplemente agrega un nuevo modelo y en la parte de JavaScript toca condiciones que ya estaban creadas por Como punto extra, agregar esta funcionalidad en otro módulo separado puede traer problemas que suele haber con módulos que no mucha gente utiliza: cuando sale una nueva versión de Odoo se atrasa la migración del mismo por falta de approvals o porque nadie se pone a migrarlo. Estando todo en el mismo módulo, que es utilizado por toda la gente de la localización española, no debería de existir este problema de atrasos en la migración del mismo. |
|
No seas más papista que el Papa, Aritz. Puestos así, el |
|
Concuerdo con @pedrobaeza en que debe ser una funcionalidad separada. Nosotros, por ejemplo, no necesitamos esta funcionalidad para nada. Sencillamente nunca abrimos la misma sesión en dos dispositivos. En cualquier instalación que tenga que gestionar preferiría que no estuviera ni siquiera disponible/instalable: una cosa menos que pueda problemas. |
Keep as simple as posibleEste módulo se hizo para una pequeña tienda de Madrid de la Calle Goya en el año 2015 #170 Cuanto menos código se le ponga a un pequeño cliente, mejor. Aquellos más grandes o de más volumen con más complejidad no tenéis problemas de agregar otros módulos. Abogo por un divide y vencerás 😄 (en varios módulos) ¡Muchas gracias a todos! |
|
Buenas, he intentado instalarlo en una base de datos de test junto con otros módulos instalados y me ha dado un error de conflicto con un módulo del repo |
ao-landoo
force-pushed
the
14.0-imp-l10n_es_pos-seq_by_device
branch
from
e654361 to
de9ddc6
Compare
ao-landoo
force-pushed
the
14.0-imp-l10n_es_pos-seq_by_device
branch
3 times, most recently
from
71040e6 to
fedf92c
Compare
|
He realizado unos cambios en l10n_es_pos para facilitar la herencia desde el módulo l10n_es_pos_by_device que he creado, pero sin tocar la lógica. A ver que os parece, yo he realizado pruebas y me está funcionando bien. |
ao-landoo
force-pushed
the
14.0-imp-l10n_es_pos-seq_by_device
branch
from
fedf92c to
15a70b6
Compare
|
@pedrobaeza @rafaelbn @blaskurain @Mat-moran ¿De esta manera todos contentos? |
|
Feedback de alguno? Nos convendría desbloquar esto para llevar l10n_es_pos a la versión 16 lo antes posible. |
|
Yo no tengo cliente del POS en 14 para probar y valorar. |
|
Yo más modificaciones las veo bien, me parece correcto haberlo separado y probado en el runbot no da fallo, por mi parte se puede hacer el merge para que se pueda seguir con la migración a la v16 |
|
/ocabot merge patch |
|
On my way to merge this fine PR! |
|
Congratulations, your PR was merged at 2ffd2cc. Thanks a lot for contributing to OCA. ❤️ |
|
¿Quién lo llevará a 15 y 16? Como comenté, por el momento no tengo ningún POS 😄 @ae-landoo puedes hacer tu el forwardport? Muchísimas gracias de ❤️ |
Ya me encargo yo, a ver si saco un rato algún día de estos |
|
Gracias @ae-landoo @pedrobaeza propongo por favor planificar para el Code Sprint ampliar la cobertura de TEST de este módulo, puede ser interesante de cara a su mantenimiento |
|
@rafaelbn gracias por los créditos pero es mi compañero @ao-landoo el que está haciendo el trabajo. |
Dejo el PR en el que se agrega la nueva funcionalidad al módulo del POS español para permitir usuarios concurrentes desde diferentes dispositivos en la misma sesión de POS.
Dejar claro que es una funcionalidad opcional, y si no se activa desde ajustes todo sigue igual. He dejado las instrucciones en el README, pero básicamente se trata de: