Skip to content

Editorial review: Add speculationrules request.destination#40357

Merged
hamishwillee merged 4 commits into
mdn:mainfrom
chrisdavidmills:add-speculationrules-request-destination
Jul 16, 2025
Merged

Editorial review: Add speculationrules request.destination#40357
hamishwillee merged 4 commits into
mdn:mainfrom
chrisdavidmills:add-speculationrules-request-destination

Conversation

@chrisdavidmills

@chrisdavidmills chrisdavidmills commented Jul 14, 2025

Copy link
Copy Markdown
Contributor

Description

As requested in #40289, this PR adds information to MDN about the speculationrules request destination type.

See https://wicg.github.io/nav-speculation/speculation-rules.html#fetch for the specification.

Browser compat information TBC.

Motivation

Additional details

Related issues and pull requests

Fixes #40289.

@chrisdavidmills chrisdavidmills requested review from a team as code owners July 14, 2025 15:00
@chrisdavidmills chrisdavidmills requested review from hamishwillee and wbamberg and removed request for a team July 14, 2025 15:00
@github-actions github-actions Bot added Content:WebAPI Web API docs Content:HTTP HTTP docs size/s [PR only] 6-50 LoC changed labels Jul 14, 2025
@chrisdavidmills chrisdavidmills changed the title Add speculationrules request.destination Technical review: Add speculationrules request.destination Jul 14, 2025
@chrisdavidmills

Copy link
Copy Markdown
Contributor Author

@domenic, do you think this needs anything else?

@github-actions

github-actions Bot commented Jul 14, 2025

Copy link
Copy Markdown
Contributor

Preview URLs

Flaws (3)

Note! 1 document with no flaws that don't need to be listed. 🎉

URL: /en-US/docs/Web/HTTP/Reference/Headers/Speculation-Rules
Title: Speculation-Rules header
Flaw count: 3

  • unknown:
    • No generic content config found
    • no blog root
    • no blog root

(comment last updated: 2025-07-16 00:10:39)

@domenic domenic left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Looks great!

Comment thread files/en-us/web/http/reference/headers/speculation-rules/index.md Outdated
@chrisdavidmills chrisdavidmills changed the title Technical review: Add speculationrules request.destination Editorial review: Add speculationrules request.destination Jul 15, 2025
@chrisdavidmills

Copy link
Copy Markdown
Contributor Author

Technical review done, cheers @domenic.

@hamishwillee can I get you to sign off on this one?

@hamishwillee hamishwillee left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Noice. Thank you.

@hamishwillee hamishwillee merged commit abdfc17 into mdn:main Jul 16, 2025
8 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Content:HTTP HTTP docs Content:WebAPI Web API docs size/s [PR only] 6-50 LoC changed

Projects

None yet

Development

Successfully merging this pull request may close these issues.

"speculationrules" request destination missing

4 participants