Skip to content

[18.0][MIG] pos_quick_logout: Migration to 18.0 - #5

Open
P-H-Phuc wants to merge 20 commits into
18.0from
18.0-mig-pos_quick_logout
Open

[18.0][MIG] pos_quick_logout: Migration to 18.0#5
P-H-Phuc wants to merge 20 commits into
18.0from
18.0-mig-pos_quick_logout

Conversation

@P-H-Phuc

@P-H-Phuc P-H-Phuc commented Jul 18, 2025

Copy link
Copy Markdown
Owner

This module add a simple button 'Log Out' available in Point Of Sale Front End UI. This button is available when the cashier is not the initial user who open session in Odoo.

The cashier is initial user:

pos_quick_logout_no_button

The cashier is not initial user:

 pos_quick_logout_with_button

legalsylvain and others added 19 commits July 14, 2025 14:21
OCA Transbot updated translations from Transifex

OCA Transbot updated translations from Transifex

OCA Transbot updated translations from Transifex

[UPD] Update pos_quick_logout.pot
[IMP] it's better to launch timeout fonction with start
[FIX] Fix Travis

[FIX] Fix Licence

[FIX] rename pos_quick_logout.py to pos_config.py

[FIX] Travis

[FIX] fix javascript syntax + __manifest__ typo

[FIX] fix python syntax

[FIX] rename pos_quick_logout_view.xml to pos_config_view.xml

[FIX] Travis : remove unused imports
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate.

Translation: pos-12.0/pos-12.0-pos_quick_logout
Translate-URL: https://translation.odoo-community.org/projects/pos-12-0/pos-12-0-pos_quick_logout/

Translated using Weblate (Spanish)

Currently translated at 100.0% (5 of 5 strings)

Translation: pos-12.0/pos-12.0-pos_quick_logout
Translate-URL: https://translation.odoo-community.org/projects/pos-12-0/pos-12-0-pos_quick_logout/es/
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate.

Translation: pos-13.0/pos-13.0-pos_quick_logout
Translate-URL: https://translation.odoo-community.org/projects/pos-13-0/pos-13-0-pos_quick_logout/
Currently translated at 100.0% (5 of 5 strings)

Translation: pos-13.0/pos-13.0-pos_quick_logout
Translate-URL: https://translation.odoo-community.org/projects/pos-13-0/pos-13-0-pos_quick_logout/ca/
Currently translated at 100.0% (5 of 5 strings)

Translation: pos-13.0/pos-13.0-pos_quick_logout
Translate-URL: https://translation.odoo-community.org/projects/pos-13-0/pos-13-0-pos_quick_logout/it/

Translated using Weblate (Italian)

Currently translated at 100.0% (5 of 5 strings)

Translation: pos-13.0/pos-13.0-pos_quick_logout
Translate-URL: https://translation.odoo-community.org/projects/pos-13-0/pos-13-0-pos_quick_logout/it/
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate.

Translation: pos-13.0/pos-13.0-pos_quick_logout
Translate-URL: https://translation.odoo-community.org/projects/pos-13-0/pos-13-0-pos_quick_logout/
Currently translated at 100.0% (5 of 5 strings)

Translation: pos-13.0/pos-13.0-pos_quick_logout
Translate-URL: https://translation.odoo-community.org/projects/pos-13-0/pos-13-0-pos_quick_logout/it/

@thienvh332 thienvh332 left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Functionally, according to the description, it’s good.

However, based on the code, you’ve removed the idle auto-logout feature. Does Odoo natively support this?

<xpath expr="//div[hasclass('oe_status')]" position="after">
<button
id="pos-quick-logout"
class="btn btn-light btn-lg lh-lg"

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Suggested change
class="btn btn-light btn-lg lh-lg"
class="btn btn-light btn-lg lh-lg"
title="Logout"

Copy link
Copy Markdown
Owner Author

Choose a reason for hiding this comment

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

In Odoo 18.0, listening event based on user interface (keypress, mousedown, onclick,...) will show respectively screen respectively
idleTimeout
useIdleTimer

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Please add that information to the PR description 😉

@P-H-Phuc
P-H-Phuc force-pushed the 18.0-mig-pos_quick_logout branch from c6bf414 to 5d98754 Compare August 11, 2025 03:35

@thienvh332 thienvh332 left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pre-approved

@thienvh332

thienvh332 commented Aug 18, 2025

Copy link
Copy Markdown

@P-H-Phuc ,
I think this is ready to open a PR on OCA, right?

@P-H-Phuc

Copy link
Copy Markdown
Owner Author

@P-H-Phuc , I think this is ready to open a PR on OCA, right?

Thank you, I created PR on OCA

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.