Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

[MIG][12.0] mig pos_customer_display #437

Merged
merged 48 commits into from May 19, 2020
Merged

[MIG][12.0] mig pos_customer_display #437

merged 48 commits into from May 19, 2020

Conversation

legalsylvain
Copy link
Contributor

@legalsylvain legalsylvain commented Jan 9, 2020 •
edited

migrate pos_customer_display from 10.0 to 12.0

[FIX] bugs presents in previous version, that was raising messages many times.
[REF] split Js file in many files, following odoo convention
[REF] OCA convention (readme, etc...)
[ADD] format field (for instance only 1 option 2 lines x 20 characters) and add specific JS file to generate message for this format. It will allow later to implement other format, specially the format 4 lines x 30 characters, of the Bixolon BCD-2000)
[ADD] message when the customer account is selected.
[IMP] display also discount, if any.
[FIX] make default message translatable ("welcome", "PoS Open", etc...)

#GRAPOCA

Auélien DUMAINE and others added 30 commits January 9, 2020 09:52
@legalsylvain
Initial check-in of the module pos_customer_display.
d0f5463
@legalsylvain
Initial check-in of the module pos_payment_terminal.
f324452
@legalsylvain
pos_customer_display bugfix : new massage on deleting orderLine and f…
a6af3c8 
…ixing unit_price computation on adding orderLine
@legalsylvain
Fix copyright headers
d0f07cf 
PEP8 stuff
@legalsylvain
pos_payment_terminal : bigger button for start transaction with touch…
52b0d8c 
…screen
@legalsylvain
Add french translation
e9c2313
@legalsylvain
FIX Adapt JS code to recent changes in the code of POS v8
4756937 
Update module description with my recent tests with new hardware
@legalsylvain
pos_payment_terminal: add missing css file
9ab59c3 
pos_customer_display: FIX JS code and make it more robust
@legalsylvain
[FIX] lenght -> length
1dc4ca8 
[IMP] Change is now displayed on the LCD when entered (avoid to keep displaying the change with the "next customer")
Rename function names to easier understanding
@legalsylvain
FIX bugs
bf8560b 
Remove paiment method from LCD message
Add missing POT file
Update FR translation
@legalsylvain
Add message on POS open
564bd50 
Remove code that was not working
FIX bug in removeOrderLine
Update translation
@legalsylvain
Add Spanish translation by Alejandro Amador (thanks for your contribu…
b7a31a8 
…tion !)

Move CSS definition from pos_payment_terminal_view.xml to pos_payment_terminal.xml
Small cleanups
@legalsylvain
Update ES translation by Alejandro Amador
5fab8d8
@legalsylvain
PEP8
c8324aa
@yvaucher @legalsylvain
Add license key in __openerp__.py
50edb53
@gurneyalex @legalsylvain
Add OCA as author of OCA addons
01bb345 
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.
@JosDeGraeve @legalsylvain
Added dutch translation
a08d670
@legalsylvain
[IMP] Add German Language
7f70219
@legalsylvain
[FIX] correcting some chars
0265c0d
@legalsylvain
pos_customer_display: Add button "Display Total to Customer" and port…
eae6096 
… to new API

pos_payment_terminal: code cleanup
@alexis-via @legalsylvain
Cleaner inheritance implementation in pos_customer_display
40f2edc 
Better debug logs in hw_* modules for LCD and Telium
@alexis-via @legalsylvain
pos_customer_display: finish to convert all JS inherits to cleaner im…
bbee581 
…plementation
@alexis-via @legalsylvain
Remove some console.log for debug
d9c95a2
@alexis-via @legalsylvain
Make some messages of customer display configurable
b359486 
JS code cleanup: harmonize identation
@alexis-via @legalsylvain
PEP8
9d6691a
@alexis-via @legalsylvain
FIX when clicking on "Display Total to Customer", the query is now se…
032a179 
…nt only once to the POSbox

Patch written with the kind help of Sylvain Calador
@sbidoul @legalsylvain
[UPD] prefix versions with 8.0
faa616b
@pedrobaeza @legalsylvain
[MIG] Make modules uninstallable
a8fc9c3
@pedrobaeza @legalsylvain
[MIG] Rename manifest files
08c4b72
@wamdam @legalsylvain
Make pos_customer_display odoo9 compatible
e5bcc84
alexis-via and others added 15 commits January 9, 2020 09:52
@alexis-via @legalsylvain
Fix version number of pos_customer_display
0f470d5
@alexis-via @legalsylvain
Port pos_customer_display to v10
3b13c7f 
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
@alexis-via @legalsylvain
Display relevant info on LCD when Qty/price/discount is forced via th…
ebdf5a4 
…e buttons of the POS frontend
@alexis-via @legalsylvain
Remove comment
901568d
@oca-transbot @legalsylvain
OCA Transbot updated translations from Transifex
4e6fcbb
@oca-transbot @legalsylvain
OCA Transbot updated translations from Transifex
4cbc497
@oca-transbot @legalsylvain
OCA Transbot updated translations from Transifex
a6e532f
@gaelTorrecillas @legalsylvain
[fix] compability between pos_customer_display and pos_pricelist (OCA…
faf1c61 
…#210)

* [fix] compability between pos_customer_display and pos_pricelist
@legalsylvain
[FIX] Fix pos_customer_display with Floors in Restaurant
6fb4375
@oca-transbot @legalsylvain
OCA Transbot updated translations from Transifex
3413636
@chienandalu @legalsylvain
[CI] Fix .rst lint errors (OCA#276)
1e30883 
* [CI] Fix .rst lint errors
* [FIX] licence Key
@sbidoul @legalsylvain
remove obsolete .pot files [ci skip]
041cea6
@oca-travis @legalsylvain
[UPD] Update pos_customer_display.pot
8f3e8a7
@OCA-git-bot @legalsylvain
[ADD] icon.png
56bdbcc
@legalsylvain
[REF] pos_customer_display: Black python code
41a612e
@legalsylvain legalsylvain added this to the 12.0 milestone Jan 9, 2020
@OCA-git-bot OCA-git-bot mentioned this pull request Jan 9, 2020
Closed
13 tasks
@legalsylvain
[MIG] pos_customer_display: Migration to 12.0
07737ab 
[REF] OCA Convention
- rename files.
- split js file into many files. (models, screens, etc.)
[FIX] make default "welcome" and "good bye" message translatable
[REF] add the possibility to have other format of LCD Customer display.

fixup! fixup! fixup! fixup! fixup! fixup! fixup! fixup! [MIG] pos_customer_display: Migration to 12.0 [REF] OCA Convention. rename files.

fixup! fixup! fixup! fixup! fixup! fixup! fixup! fixup! fixup! [MIG] pos_customer_display: Migration to 12.0 [REF] OCA Convention. rename files.

fixup! fixup! fixup! fixup! fixup! fixup! fixup! fixup! fixup! fixup! [MIG] pos_customer_display: Migration to 12.0 [REF] OCA Convention. rename files.

fixup! fixup! fixup! fixup! fixup! fixup! fixup! fixup! fixup! fixup! fixup! [MIG] pos_customer_display: Migration to 12.0 [REF] OCA Convention. rename files.

fixup! fixup! fixup! fixup! fixup! fixup! fixup! fixup! fixup! fixup! fixup! fixup! [MIG] pos_customer_display: Migration to 12.0 [REF] OCA Convention. rename files.

fixup! fixup! fixup! fixup! fixup! fixup! fixup! fixup! fixup! fixup! fixup! fixup! fixup! [MIG] pos_customer_display: Migration to 12.0 [REF] OCA Convention. rename files.
@legalsylvain
fixup! [MIG] pos_customer_display: Migration to 12.0 [REF] OCA Conven…
e47f1fe 
…tion - rename files. - split js file into many files. (models, screens, etc.) [FIX] make default "welcome" and "good bye" message translatable [REF] add the possibility to have other format of LCD Customer display.
@legalsylvain
fixup! fixup! [MIG] pos_customer_display: Migration to 12.0 [REF] OCA…
d28e812 
… Convention - rename files. - split js file into many files. (models, screens, etc.) [FIX] make default "welcome" and "good bye" message translatable [REF] add the possibility to have other format of LCD Customer display.
@legalsylvain legalsylvain changed the title [MIG][12.0] mig pos_customer_display (WIP) [MIG][12.0] mig pos_customer_display Jan 12, 2020
@legalsylvain
Copy link
Contributor Author

CC : @quentinDupont, @alexis-via, @sebastienbeau.

@legalsylvain
Copy link
Contributor Author

CC : @PierrickBrun

@quentinDupont
Copy link

migrate pos_customer_display from 10.0 to 12.0

[FIX] bugs presents in previous version, that was raising messages many times.
[REF] split Js file in many files, following odoo convention
[REF] OCA convention (readme, etc...)
[ADD] format field (for instance only 1 option 2 lines x 20 characters) and add specific JS file to generate message for this format. It will allow later to implement other format, specially the format 4 lines x 30 characters, of the Bixolon BCD-2000)
[ADD] message when the customer account is selected.
[IMP] display also discount, if any.
[FIX] make default message translatable ("welcome", "PoS Open", etc...)

#GRAPOCA

seems very promising, i have to test it as soon as possible :)

@alexis-via alexis-via merged commit 0bb73a5 into OCA:12.0 May 19, 2020
@legalsylvain
Copy link
Contributor Author

Hi @alexis-via. Thanks for your review.

The next time you want to merge a PR could you use oca-bot command ? It makes test and update readme file.
Thanks !

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

@alexis-via alexis-via alexis-via approved these changes

@PierrickBrun PierrickBrun PierrickBrun approved these changes

Assignees
No one assigned
Labels
approved
Projects
None yet
Milestone
12.0
Development

Successfully merging this pull request may close these issues.

None yet