Skip to content

fix: implement ping-pong mechanism to keep service worker alive #1956

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

Draft
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

mkazlauskas
Copy link
Member

The service worker was becoming inactive when browser tabs lost focus. Implemented a more reliable ping-pong mechanism that sends regular ping messages from all UI contexts to the background service, preventing the browser from considering the service worker inactive.

Checklist

  • JIRA - <link>
  • Proper tests implemented
  • Screenshots added.

Proposed solution

Explain how does this PR solves the problem stated in JIRA ticket.
You can also enumerate different alternatives considered while approaching this task.

Testing

Untested

Screenshots

Attach screenshots here if implementation involves some UI changes

@mkazlauskas mkazlauskas requested a review from ljagiela July 21, 2025 14:49
@mkazlauskas mkazlauskas requested a review from a team as a code owner July 21, 2025 14:49
@mkazlauskas mkazlauskas marked this pull request as draft July 21, 2025 14:50
The service worker was becoming inactive when browser tabs lost focus.
Implemented a more reliable ping-pong mechanism that sends regular ping
messages from all UI contexts to the background service, preventing the
browser from considering the service worker inactive.
Copy link

@lace-bot
Copy link
Collaborator

Allure Report

allure-report-publisher generated test report!

processReports: ✅ test report for aae4e451

passed failed skipped flaky total result
Total 33 0 4 0 37

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.

2 participants