Skip to content
This repository has been archived by the owner on Sep 29, 2024. It is now read-only.

Demo is broken #3

Open
mocanuga opened this issue Jun 14, 2018 · 1 comment
Open

Demo is broken #3

mocanuga opened this issue Jun 14, 2018 · 1 comment

Comments

@mocanuga
Copy link

The demo is broken in Edge/17.17134
It seems that the whole service worker registration process is broken.

(function (win, nav) {
    'use strict';
    var pushNotificationsSupport = 'serviceWorker' in nav && 'PushManager' in win && 'Notification' in win;
    console.log('push notification suppport: ', pushNotificationsSupport)
    if(pushNotificationsSupport) {
	nav.serviceWorker.register('/tests/ie/sw.js', {scope: '/tests/ie/'})
		.then(function() {
			console.log(arguments);
		})
		.catch(function(err) {
			console.log(err);
		})
    }
}(window, navigator));

The code above fails with
{description: "Invalid argument", message: "Invalid argument", number: -2147418113}

@neuhaus
Copy link

neuhaus commented Jan 27, 2023

I came here to report that the demo is still broken as of Jan 27th 2023.
I tried https://webpushdemo.azurewebsites.net/ on macOS 13.2 with Firefox 109, Chromium 110 and Safari 16.3. Subscribing seems to work, but after closing the tab and waiting for the popup notification, it never arrives.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants