12.0 mig pos customer display#5
Conversation
Update description Remove test on number of row/cols, because it is already tested on POS client JS code.
PEP8 stuff
Update module description with my recent tests with new hardware
Adapt JS code of pos_payment_terminal Some cleanup
In order to get visibility on https://www.odoo.com/apps the OCA board has decided to add the OCA as author of all the addons maintained as part of the association.
Better debug logs in hw_* modules for LCD and Telium
…d flake8 on pos_remove_pos_category
…d flake8 on pos_remove_pos_category
…os_remove_pos_category
…ixing unit_price computation on adding orderLine
PEP8 stuff
Update module description with my recent tests with new hardware
pos_customer_display: FIX JS code and make it more robust
…nt only once to the POSbox Patch written with the kind help of Sylvain Calador
Remove the "Show total to customer" button, which is not need given the new user interface of the POS: the big "Payment" button has this role now. Add README.rst
…e buttons of the POS frontend
…#210) * [fix] compability between pos_customer_display and pos_pricelist
* [CI] Fix .rst lint errors * [FIX] licence Key
vvrossem
force-pushed
the
12.0-mig-pos_customer_display
branch
from
e38f7da to
731a0e0
Compare
|
@vvrossem pour les copyrights, en effet, le pavé sur AGPL n'est plus nécessaire. Il faut laisser les copyrights déjà présents et ajouter celui de Coop IT Easy en dessous. par exemple ce module doit avoir tout ce dont tu peux avoir besoin https://github.com/OCA/pos/tree/12.0/pos_access_right. Pour le bouton d'Odoo, pourquoi ne pas l'utiliser directement plutôt que d'en créer un second ? |
There was a problem hiding this comment.
Pas mal de taff, quelque questions sinon ça m'a l'air bon.
Je n'ai pas encore suivi en détail les recommandations OCA vers 11.0 et 12.0
Ok, pas besoin de tout suivre à la lettre, surtout tant qu'on ne soumet pas à l'OCA
Côté JS, la fonction initialize dans models.PosModel permet d'établir la connexion avec le proxy lorsque Customer Display Device Management est coché
Questions :
Que faire des divers copyrights présents et où intégrer le notre?
Préserver ceux présents et rajouter, par exemple:
# Copyright (C) 2014 Akretion (http://www.akretion.com)
# Alexis de Lattre <alexis.delattre@akretion.com>
# Copyright 2019 Coop IT Easy SCRLfs
# Robin Keunen <robin@coopiteasy.be>
# License AGPL-3.0 or later (http://www.gnu.org/licenses/agpl.html).
Est-ce que l'écran de configuration ci-dessous retire la possible confusion avec le field Customer Display développé par Odoo? Il n'est visible que lorsque PosBox est coché
Oui et j'irais même jusqu'à spécifier que c'est codé pour Bixolon
| <?xml version="1.0"?> | ||
| <odoo> | ||
|
|
||
| <!--TODO(Vincent) rename file pos_config_view?--> |
There was a problem hiding this comment.
Quand tu fais une migration, c'est mieux de laisser les fichiers tels quels même si le nom ne parait pas très pertinent. Ou, si tu le fais quand même, les renommer dans un commit à part.
|
|
||
| iface_customer_display = fields.Boolean( | ||
| default=False, string='Customer Display', help="Display data on the customer display") | ||
| string='Customer Display Device Management', help="Display data on the customer display device") |
There was a problem hiding this comment.
La ligne va être trop longue pour pep8
There was a problem hiding this comment.
Ce serait pas mal, que ton code passe les check travis :-)
|
There hasn't been any activity on this pull request in the past 4 months, so it has been marked as stale and it will be closed automatically if no further activity occurs in the next 30 days. |
16 participants
@robinkeunen
@PierrickBrun
Migration fonctionnelle du module pos_customer_display de 10.0 à 12.0.
Remarques :
Questions :