[IMP] POS: customer display config and use update #14000
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
|
|
auva-odoo
requested review from
xpl-odoo
and removed request for
a team and
auva-odoo
task-4920460
emmi-odoo
force-pushed
the
18.0-customer-display-lrpz
branch
from
bcc0f79 to
6b4fb1e
Compare
xpl-odoo
requested changes
Jul 24, 2025
Comment on lines
+15
to
+16
| Both the customer display and POS display should be at least 6 inches. However, a larger screen | ||
| is recommended for improved end-user readability. |
There was a problem hiding this comment.
Suggested change
| Both the customer display and POS display should be at least 6 inches. However, a larger screen | |
| is recommended for improved end-user readability. | |
| Both the customer and POS displays must have a minimum diagonal size of 6 inches. | |
| For optimal readability, larger screens are recommended. |
- I would specify "diagonal size" even if it's the common way to measure screens.
- "Larger screen" is quite vague. If possible, it would be helpful to recommend a specific size.
Comment on lines
+21
to
+24
| Depending on the POS setup, feature can be displayed :ref:`directly on a secondary screen | ||
| <customer_display/local>` connected via USB-C or HDMI, :ref:`on another device remotely | ||
| <customer_display/remote>`, or :ref:`on a screen connected through an IoT system | ||
| <customer_display/iot>`. |
There was a problem hiding this comment.
Suggested change
| Depending on the POS setup, feature can be displayed :ref:`directly on a secondary screen | |
| <customer_display/local>` connected via USB-C or HDMI, :ref:`on another device remotely | |
| <customer_display/remote>`, or :ref:`on a screen connected through an IoT system | |
| <customer_display/iot>`. |
I wouldn't start the configuration section with that, especially since you used refs as users might click and go to the relevant section directly, skipping the common configuration step. You could tweak the intro paragraph to include that info if you feel it's necessary.
Comment on lines
+28
to
+29
| #. Navigate to the :ref:`POS settings <configuration/settings>`. | ||
| #. Scroll down to the :guilabel:`Connected Devices` section. |
There was a problem hiding this comment.
Suggested change
| #. Navigate to the :ref:`POS settings <configuration/settings>`. | |
| #. Scroll down to the :guilabel:`Connected Devices` section. | |
| #. Navigate to the :ref:`POS settings <configuration/settings>` and scroll down to the :guilabel:`Connected Devices` section. |
Having a separate step for "Scrolling down" is overkill :)
Comment on lines
+34
to
+36
| - :guilabel:`The same device`: To use the secondary screen connected with an HDMI or USB-C cable. | ||
| - :guilabel:`Another device`: To connect to a remote device connected to the database. | ||
| - :guilabel:`IOT-connected screen` for displays connected via an :doc:`IoT system |
There was a problem hiding this comment.
You could add the refs here instead
| #. Open the dropdown menu under the :guilabel:`Customer Display` section and select one of the | ||
| following options: | ||
|
|
||
| - :guilabel:`None`: To disable the customer display. |
There was a problem hiding this comment.
Suggested change
| - :guilabel:`None`: To disable the customer display. | |
| - :guilabel:`None`: To disable the feature |
|
|
||
| #. :ref:`Open a POS session <pos/session-start>`. | ||
| #. Click the :icon:`fa-bars` icon (:guilabel:`hamburger menu`). | ||
| #. Click the :icon:`fa-desktop` icon (:guilabel:`customer screen`) to open a new window to drag and | ||
| drop onto the second screen. | ||
| #. Click the :icon:`fa-desktop` icon (:guilabel:`customer screen`), which opens a new window to drag |
There was a problem hiding this comment.
Suggested change
| #. Click the :icon:`fa-desktop` icon (:guilabel:`customer screen`), which opens a new window to drag | |
| #. Click the :icon:`fa-desktop` (:guilabel:`Customer Display`) icon, which opens a new window to drag |
I recommend using the tooltip as it is (when there is one obviously 😅)
| Access your database from another device (any computer, tablet, or smartphone), go to the POS | ||
| application, click the vertical ellipsis button (:guilabel:`⋮`) on a POS card, and then | ||
| :guilabel:`Customer Display` to open the display remotely. | ||
| To open the customer display on a remote device, (any computer, tablet, or smartphone), follow these |
There was a problem hiding this comment.
Suggested change
| To open the customer display on a remote device, (any computer, tablet, or smartphone), follow these | |
| To open the customer display on a remote device (any computer, tablet, or smartphone), follow these |
Comment on lines
+77
to
+78
| #. Access your database from the other device. | ||
| #. Navigate to :menuselection:`Point of Sale --> Dashboard`. |
There was a problem hiding this comment.
Suggested change
| #. Access your database from the other device. | |
| #. Navigate to :menuselection:`Point of Sale --> Dashboard`. | |
| #. Access your database from the other device and open the :guilabel:`Point of Sale` app. |
This can be a single step.
Comment on lines
+79
to
+80
| #. Click the :icon:`fa-ellipsis-v` icon (:guilabel:`vertical ellipsis`) on a POS card. | ||
| #. Click :guilabel:`Customer Display` to open the display remotely. |
There was a problem hiding this comment.
Suggested change
| #. Click the :icon:`fa-ellipsis-v` icon (:guilabel:`vertical ellipsis`) on a POS card. | |
| #. Click :guilabel:`Customer Display` to open the display remotely. | |
| #. Click the :icon:`fa-ellipsis-v` (:guilabel:`Dropdown menu`) icon on a POS card. | |
| #. Click :guilabel:`Customer Display` to open the display remotely. |
Comment on lines
+94
to
+96
| #. Click the :icon:`fa-bars` icon (:guilabel:`hamburger menu`). | ||
| #. Click the :icon:`fa-desktop` icon (:guilabel:`customer screen`), which opens the customer display | ||
| onto the IoT-connected display. |
There was a problem hiding this comment.
Suggested change
| #. Click the :icon:`fa-bars` icon (:guilabel:`hamburger menu`). | |
| #. Click the :icon:`fa-desktop` icon (:guilabel:`customer screen`), which opens the customer display | |
| onto the IoT-connected display. | |
| #. Click the :icon:`fa-bars` (:guilabel:`hamburger menu`) icon. | |
| #. Click the :icon:`fa-desktop` (:guilabel:`Customer Display`) icon to open the display | |
| onto the IoT-connected screen. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
task-4920460
18.3: remote config was deleted in 18.3, remove it in fw (cf. task)