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

Add 'background-sync' permission #60

Closed
nsatragno opened this issue Feb 16, 2016 · 4 comments
Closed

Add 'background-sync' permission #60

nsatragno opened this issue Feb 16, 2016 · 4 comments

Comments

@nsatragno
Copy link
Member

Users should be able to disable background sync. There is already discussion going on about what to do if permission is blocked. In the meantime, it is desirable to expose this information on the Permissions API.

I suggest adding a background-sync element to PermissionName. I am happy to provide a PR with the change.

What do you think?

@mounirlamouri
Copy link
Member

@jkarlin and @mkruisselbrink, I can't find any mention of permission in the spec and the linked issue isn't resolved. Are you moving to a permission-based API?

@mkruisselbrink
Copy link

Are you moving to a permission-based API?

At least in chrome we're likely never going to prompt for a background sync permission, but yes, we are going to add the possibility for users to disable background sync. We just haven't gotten around to updating the spec to address this possibility.
(as an aside, it doesn't seem very maintainable long term if every feature with a permission will need to update the permissions API spec... having such circular dependencies between feature and permissions API seems like it would make it impossible to move any spec forward as you don't want a "more stable" spec to depend on a "less stable" spec. Why can't specs just define their extensions to the permissions API in their own spec text?)

@beverloo
Copy link
Member

+1 to the extensions question, we'd also like to do this with push and possibly notifications.

@marcoscaceres
Copy link
Member

About extensions, it's fine for specs to do that in their own specs as a monkey patch - but as a community, we should make the effort to come back to this spec once the new permission is adopted (just file a bug).

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

5 participants