-
Notifications
You must be signed in to change notification settings - Fork 22.7k
Document fetchLater()
#39016
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
Document fetchLater()
#39016
Conversation
Preview URLs (11 pages)
Flaws (1)Note! 10 documents with no flaws that don't need to be listed. 🎉 URL:
(comment last updated: 2025-04-09 12:10:06) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@tunetheweb getting there! A few more comments for you, mostly around the quotas guide rewrite.
Co-authored-by: Chris Mills <chrisdavidmills@gmail.com>
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
@chrisdavidmills any thoughts on KB (ambiguous as to whether 1000 bytes or 1024 bytes, though technically it's 1000) versus KiB (explicitly 1024 bytes and technically more accurate)? @noamr suggested it's technically KiB and so changed it in 44b6681 but doesn't seem like KiB is used much in MDN. |
@tunetheweb I've seen KiB used elsewhere on MDN; I think it's good to be accurate. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@tunetheweb I think this is pretty much ready to go. I've just left a few non-blocking suggestions for you. Let me know when you've considered them, and I will get it merged!
@@ -2,6 +2,8 @@ | |||
title: "Permissions-Policy: deferred-fetch" | |||
slug: Web/HTTP/Reference/Headers/Permissions-Policy/deferred-fetch | |||
page-type: http-permissions-policy-directive | |||
status: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Not a blocker, but you actually don't need to add these anymore; the system adds them automatically based on the BCD.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
(just informational; you don't need to remove them again)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Or do you mean they ARE needed, but will be added automatically after merge so I didn't need to add them manually (but no harm that I have done it - but more FYI for next time to save me some bother?).
Yes, exactly correct.
Co-authored-by: Chris Mills <chrisdavidmills@gmail.com>
Co-authored-by: Chris Mills <chrisdavidmills@gmail.com>
Co-authored-by: Chris Mills <chrisdavidmills@gmail.com>
Thanks for your help reviewing @noamr and @chrisdavidmills ! I've seen some movement on the spec PR so hopefully that'll follow along soon. But this one has dragged on so let's not wait for that in case it's not as close as I think since Chrome has shipped this already. Will add spec links to BCD after it's been merged and it should flow into here. |
Nice work, @tunetheweb. |
* Fetch Later API * More updates * Rename api * Add quotas docs * Tweaks * Cleanup * Linting fix * Linting fix * Update PP docs * Fix folder name * Fix example * Linting fixes * More linting fixes * Tweak * Explain example * More linting * Add CSP and more exception details * Update files/en-us/web/api/window/fetchlater/index.md Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com> * Update files/en-us/web/api/window/fetchlater/index.md Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com> * Update files/en-us/web/api/window/fetchlater/index.md Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com> * Update files/en-us/web/api/window/fetchlater/index.md Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com> * more quota examples * Update files/en-us/web/api/fetchlater_api/fetchlater_quotas/index.md Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com> * Review feedback * Accepting first batch of review feedback Co-authored-by: Chris Mills <chrisdavidmills@gmail.com> * Second batch of suggestions from code review Co-authored-by: Chris Mills <chrisdavidmills@gmail.com> * Update files/en-us/web/api/fetchlater_api/index.md Co-authored-by: Chris Mills <chrisdavidmills@gmail.com> * Update files/en-us/web/api/fetchlater_api/index.md * Update files/en-us/web/api/fetchlater_api/index.md * Update files/en-us/web/api/fetchlater_api/index.md * Update files/en-us/web/api/fetchlater_api/fetchlater_quotas/index.md * Update files/en-us/web/api/fetchlater_api/index.md Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com> * Update files/en-us/web/api/fetchlater_api/fetchlater_quotas/index.md * Update files/en-us/web/api/fetchlater_api/fetchlater_quotas/index.md * Update files/en-us/web/api/window/fetchlater/index.md * Review feedback * Update files/en-us/web/api/fetchlaterresult/activated/index.md Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com> * Rename file * Meta cleanup * Tweaks * browser-compat for deferred-fetch and deferred-fetch-init * Permissions Policy updates * Apply suggestions from code review Co-authored-by: Chris Mills <chrisdavidmills@gmail.com> * Update files/en-us/web/api/fetchlater_api/fetchlater_quotas/index.md Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com> * Update files/en-us/web/api/fetchlater_api/fetchlater_quotas/index.md * Review feedback * KB -> KiB * Permissions Policy BCD fixes * Frontmatter * Fetch Interfaces * Update files/en-us/web/api/deferredrequestinit/index.md Co-authored-by: Chris Mills <chrisdavidmills@gmail.com> * Update files/en-us/web/api/fetchlater_api/fetchlater_quotas/index.md Co-authored-by: Chris Mills <chrisdavidmills@gmail.com> * Update files/en-us/web/api/fetchlater_api/fetchlater_quotas/index.md Co-authored-by: Chris Mills <chrisdavidmills@gmail.com> --------- Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com> Co-authored-by: Chris Mills <chrisdavidmills@gmail.com>
Description
Adds docs on
fetchLater()
API added in Chrome 135.Motivation
New API that needs documenting.
NOTE the spec changes have NOT been merged yet, so the docs show missing spec at present.
Additional details
ChromeStatus: https://chromestatus.com/feature/4654499737632768
Previous Origin Trail article on developer.chrome.com: https://developer.chrome.com/blog/fetch-later-api-origin-trial
Explainer: https://github.com/WICG/pending-beacon/blob/main/docs/fetch-later-api.md
Spec PR (not merged yet): whatwg/fetch#1647
Related issues and pull requests
Added to BCD in mdn/browser-compat-data#26135 and mdn/browser-compat-data#26437.