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.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I understand how vague "larger" is, but it really depends on how someone uses their POS, whether on their phone, tablet, or computer; it all works as long as it's at least 6 inches. I would say it's up to them; some people might feel comfortable with a 6-inch screen, while others might find it way too small.

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.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I think this could be confusing, the toggle here on the pos card says:
image
So I see dropdown menu mentioned, but there's another icon that's also a vertical elipsis but the toggle doesn't say that
image

  • the hamburger menu inspection also shows dropdown:
image

Copy link
Contributor Author

Choose a reason for hiding this comment

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

For now, I'm switching to "Dropdown menu" as suggested because title = Dropdown menu, but I still think it could be confusing, wdyt?

@LoredanaLrpz LoredanaLrpz force-pushed the 18.0-customer-display-lrpz branch from 6b4fb1e to 8d20e67 Compare July 28, 2025 16:01
@LoredanaLrpz LoredanaLrpz requested review from xpl-odoo and removed request for xpl-odoo July 28, 2025 16:02
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