Skip to content
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

Instances #1

Open
karmanyaahm opened this issue Jun 25, 2022 · 1 comment
Open

Instances #1

karmanyaahm opened this issue Jun 25, 2022 · 1 comment

Comments

@karmanyaahm
Copy link
Member

karmanyaahm commented Jun 25, 2022

Is there any way to do multiple push registrations for one service worker on web?

If not (as far as I know it's not possible), how do we handle instances in this lib?
Remove the instance parameter, becuase instances won't work in web anyway and this will cause inconsistency, or keep it for android?

Right now, everything just returns default instance, and instance checking in the example app is disabled.

( @p1gp1g @MatMaul ) what would you guys advise?

@karmanyaahm
Copy link
Member Author

karmanyaahm commented Jun 26, 2022

According to the W3C Push API document:

Each push subscription is associated with a service worker registration and a service worker registration has at most one push subscription.

I'm thinking I should just get rid of instances for this lib to make cross-platforming simple.

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

No branches or pull requests

1 participant