Skip to content

[IMP] POS: customer display config and use update #14000

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.

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

Open
wants to merge 1 commit into
base: 18.0
Choose a base branch
from

Conversation

LoredanaLrpz
Copy link
Contributor

@LoredanaLrpz LoredanaLrpz commented Jul 11, 2025

task-4920460

18.3: remote config was deleted in 18.3, remove it in fw (cf. task)

@robodoo
Copy link
Collaborator

robodoo commented Jul 11, 2025

Pull request status dashboard

@LoredanaLrpz LoredanaLrpz requested a review from auva-odoo July 11, 2025 16:21
@C3POdoo C3POdoo requested a review from a team July 11, 2025 16:22
@auva-odoo auva-odoo requested review from xpl-odoo and removed request for a team and auva-odoo July 15, 2025 15:20
@emmi-odoo emmi-odoo force-pushed the 18.0-customer-display-lrpz branch from bcc0f79 to 6b4fb1e Compare July 18, 2025 12:42
Copy link
Contributor

@xpl-odoo xpl-odoo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We should consider moving this page outside the "Shop features" section, since it can be used by bar/restaurants.

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.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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>`.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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`.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants