You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
I agree to follow the Code of Conduct that this project adheres to.
I have searched the issue tracker for a bug report that matches the one I want to file, without success.
Electron Version
26.2.1
What operating system are you using?
Windows
Operating System Version
Windows 10
What arch are you using?
x64
Last Known Working Electron version
No response
Expected Behavior
Electron has background sync disabled. Background sync is used to trigger the service worker to sync changes. It helps to sync work which was done by user when he was offline. Would be good to enable it by default. It is enabled by default in chromium and chrome.
On page with service worker registered, this snippet should produce no errors.
This issue was raised at #9802
It is unclear to me why #9802 was merged with #6697. The other issue is about Web Push, which is distinct from Background Sync.
Also, the #6697 was closed without a fix to the Background Sync problem #9802 .
Actual Behavior
When run with electron, I get
Uncaught (in promise) DOMException: Background Sync is disabled.
Testcase Gist URL
No response
Additional Information
How to reproduce
Open youtube.com with electron
OR
git clone https://github.com/hokein/electron-sample-apps
cd electron-sample-apps
npm install -g electron
electron service-worker/mock-response
Open the console and type in the snippet:
navigator.serviceWorker.ready.then(function(swRegistration) {
return swRegistration.sync.register('myFirstSync');
});
It gives the error in electron. Use the same snippet in youtube on the browser, it runs fine.
This is the same bug as #31063 which is closed as not planned the issue still exists with electron v 26.2.1
The text was updated successfully, but these errors were encountered:
#6697 is the correct tracking issue for this, internally they are not so distinct and the work required to support one is similar to the other (somehow exposing the ability for electron apps to provide their own service urls for push/sync)
Preflight Checklist
Electron Version
26.2.1
What operating system are you using?
Windows
Operating System Version
Windows 10
What arch are you using?
x64
Last Known Working Electron version
No response
Expected Behavior
Electron has background sync disabled. Background sync is used to trigger the service worker to sync changes. It helps to sync work which was done by user when he was offline. Would be good to enable it by default. It is enabled by default in chromium and chrome.
On page with service worker registered, this snippet should produce no errors.
This issue was raised at #9802
It is unclear to me why #9802 was merged with #6697. The other issue is about Web Push, which is distinct from Background Sync.
Also, the #6697 was closed without a fix to the Background Sync problem #9802 .
Actual Behavior
When run with electron, I get
Uncaught (in promise) DOMException: Background Sync is disabled.
Testcase Gist URL
No response
Additional Information
How to reproduce
Open youtube.com with electron
OR
git clone https://github.com/hokein/electron-sample-apps
cd electron-sample-apps
npm install -g electron
electron service-worker/mock-response
Open the console and type in the snippet:
navigator.serviceWorker.ready.then(function(swRegistration) {
return swRegistration.sync.register('myFirstSync');
});
It gives the error in electron. Use the same snippet in youtube on the browser, it runs fine.
This is the same bug as #31063 which is closed as not planned the issue still exists with electron v 26.2.1
The text was updated successfully, but these errors were encountered: