Skip to content

[19.0][MIG] web_session_auto_close: Migration to 19.0#3472

Open
Vang-NguyenPhu wants to merge 10 commits intoOCA:19.0from
komit-consulting:19.0-mig-web_session_auto_close
Open

[19.0][MIG] web_session_auto_close: Migration to 19.0#3472
Vang-NguyenPhu wants to merge 10 commits intoOCA:19.0from
komit-consulting:19.0-mig-web_session_auto_close

Conversation

@Vang-NguyenPhu
Copy link
Contributor

@Vang-NguyenPhu Vang-NguyenPhu commented Mar 11, 2026

I have:

  • Refactored the code structure so that it would be easier to extend the function.
  • Created tests to cover python code.

Laurent Stukkens and others added 9 commits March 11, 2026 14:30
This commit adds the web_session_auto_close module which purpose is
to is to automatically close the session when the last odoo tab is
closed or when the computer gets in idle mode.

The `timeout` occurs 15 seconds after the tab has been closed or
the computer got idle.
Currently translated at 100.0% (2 of 2 strings)

Translation: web-18.0/web-18.0-web_session_auto_close
Translate-URL: https://translation.odoo-community.org/projects/web-18-0/web-18-0-web_session_auto_close/it/
The previous time was ridiculously small, affecting also any test on
runboat where it's installed by default, but also not being a real case
for anyone having to start session again after 15 seconds of inactivity.

Let's adjust it to a more reasonable 10 minutes delay.
@Vang-NguyenPhu Vang-NguyenPhu force-pushed the 19.0-mig-web_session_auto_close branch from 14d5de5 to 404a6a8 Compare March 11, 2026 08:00
@Vang-NguyenPhu Vang-NguyenPhu mentioned this pull request Mar 13, 2026
44 tasks
Copy link

@huan-tq huan-tq left a comment

Choose a reason for hiding this comment

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

Function tested and it's LGTM!

@Quan-nhm
Copy link

I'm not sure if anyone has encountered this, but if you change the value in the system parameter to a string, this error will be raised when you refresh the page afterward.

image

@Vang-NguyenPhu Vang-NguyenPhu force-pushed the 19.0-mig-web_session_auto_close branch from 404a6a8 to ed73eb3 Compare March 13, 2026 09:39
@Vang-NguyenPhu
Copy link
Contributor Author

I'm not sure if anyone has encountered this, but if you change the value in the system parameter to a string, this error will be raised when you refresh the page afterward.

image

Thanks for pointing out, I've fixed the issue. Please help to check again, thank you.

@Vang-NguyenPhu Vang-NguyenPhu force-pushed the 19.0-mig-web_session_auto_close branch from ed73eb3 to 637885e Compare March 13, 2026 09:58
@Vang-NguyenPhu Vang-NguyenPhu force-pushed the 19.0-mig-web_session_auto_close branch from 637885e to dd5ddf0 Compare March 13, 2026 10:03
Copy link

@Quan-nhm Quan-nhm left a comment

Choose a reason for hiding this comment

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

LGTM

@hoattk-komit
Copy link

LGTM

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

8 participants