Skip to content

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.

@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
@LoredanaLrpz LoredanaLrpz force-pushed the 18.0-customer-display-lrpz branch from 8d20e67 to b0bb0df Compare August 4, 2025 18:05
@LoredanaLrpz LoredanaLrpz marked this pull request as draft August 5, 2025 09:33
@LoredanaLrpz LoredanaLrpz changed the title [IMP] POS: customer display config and use update [MOV] POS: customer display section moved + config and use update Aug 5, 2025
@LoredanaLrpz LoredanaLrpz marked this pull request as ready for review August 5, 2025 10:36
@C3POdoo C3POdoo requested a review from a team August 5, 2025 10:37
@LoredanaLrpz LoredanaLrpz force-pushed the 18.0-customer-display-lrpz branch from b0bb0df to 1e7cc01 Compare August 5, 2025 10:43
@LoredanaLrpz LoredanaLrpz requested a review from xpl-odoo August 5, 2025 12:35
@LoredanaLrpz LoredanaLrpz force-pushed the 18.0-customer-display-lrpz branch 2 times, most recently from 3f2cf07 to ae96c14 Compare August 8, 2025 06:54
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.

Just a couple suggestions :)


To activate the feature, go to the POS settings, scroll down to the :guilabel:`Connected Devices`
section, and tick the :guilabel:`Customer Display` checkbox.
For displays connected using an :doc:`IoT system <../../../general/iot>`:
Copy link
Contributor

Choose a reason for hiding this comment

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

On Odoo 18, on a trial DB (not the runbot), if you configure everything for the first time, you have to:

  • Enable IoT Box
  • Save
  • Select an already "configured" Connected Display (insert a ref here or explain?)
  • Then, and only then, you can select the "An IOT-connected screen" under Customer Display.

Otherwise, you get this error:
image

@LoredanaLrpz LoredanaLrpz force-pushed the 18.0-customer-display-lrpz branch from ae96c14 to 9e3c376 Compare August 18, 2025 15:11
task-4920460

The customer display section is now part of a bigger pos_hardware.rst documentation
page
@LoredanaLrpz LoredanaLrpz force-pushed the 18.0-customer-display-lrpz branch from 9e3c376 to 3d8ed63 Compare August 19, 2025 07:05
@LoredanaLrpz LoredanaLrpz requested a review from xpl-odoo August 19, 2025 07:06
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.

Thank you @LoredanaLrpz ! Feel free to r+ (not doing it myself in case you want to have one last look ;))

@LoredanaLrpz
Copy link
Contributor Author

@robodoo r+

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