[IMP] pos: Update ePos printers/SSL certificate doc 19.0#16680
[IMP] pos: Update ePos printers/SSL certificate doc 19.0#16680
Conversation
|
|
emmi-odoo
force-pushed
the
19.0-pos-printers-emmi
branch
4 times, most recently
from
aaada90 to
d2dd7d7
Compare
|
Hello, Here are the sources I used to write the documentation updates:
Let me know if you need any additional information, Thanks in advance for the review 😄 |
auva-odoo
left a comment
auva-odoo
left a comment
There was a problem hiding this comment.
Thank you for your work @emmi-odoo. In addition to my review comments, could you please replace all of the curly quotes in this file with straight quotes. This often happens when copying content from external sources like AI tools, so it's worth double-checking for that next time 😉
|
|
||
| For a long-term solution, you must generate a **self-signed certificate**. Then, export and import | ||
| it into your browser. | ||
| Printers that operate without an :doc:`IoT box </applications/general/iot/iot_box>` still require |
There was a problem hiding this comment.
why not IoT system (and the related link)?
content/applications/sales/point_of_sale/hardware_network/epos_ssc.rst
Outdated
Show resolved
Hide resolved
content/applications/sales/point_of_sale/hardware_network/epos_ssc.rst
Outdated
Show resolved
Hide resolved
| - Printers that use an :doc:`IoT box </applications/general/iot/iot_box>` do not need a | ||
| self-signed certificate, as the IoT box generates it automatically. |
There was a problem hiding this comment.
same question as above for IoT system instead of IoT box?
There was a problem hiding this comment.
What about this previous comment ⬆️ ?
There was a problem hiding this comment.
Oh, I thought I had changed it... Done!
| When the printer connects to a network, it automatically prints a ticket with its IP address. | ||
| Once the printer and Odoo are connected, ensure their connection is secure and reliable by | ||
| generating a :ref:`self-signed certificate <pos/epos-ssc/certificate>`. Alternatively, enable | ||
| :doc:`pos_lna` to allow Point of Sale to communicate directly with the printer on the same network. |
There was a problem hiding this comment.
I'd reverse this (first talk about lna, then ssl certificates) since it seems they're now pushing lna more?
(note: in 19.1, you will have to change this sentence as lna is mentionned in the steps!)
content/applications/sales/point_of_sale/hardware_network/receipt_printers.rst
Outdated
Show resolved
Hide resolved
content/applications/sales/point_of_sale/hardware_network/receipt_printers.rst
Outdated
Show resolved
Hide resolved
content/applications/sales/point_of_sale/hardware_network/receipt_printers.rst
Outdated
Show resolved
Hide resolved
content/applications/sales/point_of_sale/hardware_network/receipt_printers.rst
Outdated
Show resolved
Hide resolved
emmi-odoo
force-pushed
the
19.0-pos-printers-emmi
branch
2 times, most recently
from
3229e44 to
8dac6a8
Compare
emmi-odoo
force-pushed
the
19.0-pos-printers-emmi
branch
2 times, most recently
from
4f76513 to
c46ba17
Compare
auva-odoo
left a comment
auva-odoo
left a comment
There was a problem hiding this comment.
@emmi-odoo as discussed, I would review the epos_ssc.rst file to have separate sections for the different steps (so sorry about this!).
And then I have a few minor comments for receipt_printers.rst. Thank you 🙏
content/applications/sales/point_of_sale/hardware_network/receipt_printers.rst
Outdated
Show resolved
Hide resolved
| - Printers that use an :doc:`IoT box </applications/general/iot/iot_box>` do not need a | ||
| self-signed certificate, as the IoT box generates it automatically. |
There was a problem hiding this comment.
What about this previous comment ⬆️ ?
content/applications/sales/point_of_sale/hardware_network/receipt_printers.rst
Outdated
Show resolved
Hide resolved
content/applications/sales/point_of_sale/hardware_network/receipt_printers.rst
Outdated
Show resolved
Hide resolved
emmi-odoo
force-pushed
the
19.0-pos-printers-emmi
branch
3 times, most recently
from
4ef025a to
bea3650
Compare
emmi-odoo
force-pushed
the
19.0-pos-printers-emmi
branch
from
bea3650 to
4bdf99c
Compare
emmi-odoo
force-pushed
the
19.0-pos-printers-emmi
branch
from
4bdf99c to
185ee69
Compare
auva-odoo
left a comment
auva-odoo
left a comment
There was a problem hiding this comment.
Thank you very much for taking my suggestions into account @emmi-odoo and implementing changes.
@robodoo r+
task-5085342 closes #16680 Signed-off-by: Audrey Vandromme (auva) <auva@odoo.com>
3 participants
task-5085342