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 periodic-background-sync enum and description. #202

Closed
wants to merge 3 commits into from

Conversation

mugdhalakhani
Copy link

@mugdhalakhani mugdhalakhani commented Nov 7, 2019

I'm not exactly sure where the [web-background-sync] link is coming from, but I think unless there's a spec for Periodic Background Sync, I can't link to that from here. Chicken and egg. PTAL!

@beverloo FYI


Preview | Diff

@mugdhalakhani
Copy link
Author

@mounirlamouri PTAL.

index.bs Outdated
<p>
The <dfn for="PermissionName" enum-value>"periodic-background-sync"</dfn>
permission is the permission associated with the usage of
[[web-background-sync]].
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

If Periodic Background Sync is an API, you should link to it, not to Background Sync. Otherwise, reading this text, it sounds that there are two permissions meant to control the same API.

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Done. Although I'm not able to directly connect to the new spec even after updating the bibio section using:
bikeshed update --biblio

Anyway, I've added a biblio section up top to link to the PBS spec.

@@ -24,6 +24,18 @@ Mailing List Archives: http://lists.w3.org/Archives/Public/public-webappsec/

Markup Shorthands: css no, markdown yes
</pre>

<pre class="biblio">
{
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

(it will be automatically ingested by SpecRef + BikeShed in 24 hours)

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks @marcoscaceres! I'll send a PR to update the WICG/admin code.

index.bs Outdated Show resolved Hide resolved
Copy link
Member

@marcoscaceres marcoscaceres left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Just requesting to move the biblio :)

@marcoscaceres
Copy link
Member

I'm worried about adding this... I don't think we have consensus to make background sync part of the web platform. I think Mozilla it inching towards not supporting background sync, and I heard rumors that Apple was going to propose something different (some kind of d/l manager or something).

Co-Authored-By: Marcos Cáceres <marcos@marcosc.com>
@hober
Copy link
Member

hober commented Dec 10, 2019

I don't think we have consensus to make background sync part of the web platform. I think Mozilla it inching towards not supporting background sync […]

FWIW, WebKit's opposition to this feature is documented here.

@marcoscaceres
Copy link
Member

Thanks @hober for the link. I think we need to figure out what will become of periodic background sync as a spec before we proceed here.

There seems to be some renewed interest in background-fetch from multiple browser vendors instead. That's what I hear.

@mugdhalakhani
Copy link
Author

@jakearchibald, @beverloo FYI.

Base automatically changed from master to main February 16, 2021 23:20
@marcoscaceres
Copy link
Member

Given that both Mozilla and Apple were pretty firmly against this, I'm going to go ahead and close this.

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

Successfully merging this pull request may close these issues.

4 participants