Skip to content

Editorial review: Add speculationrules request.destination #40357

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

Merged

Conversation

chrisdavidmills
Copy link
Contributor

@chrisdavidmills chrisdavidmills commented Jul 14, 2025

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
Contributor Author

@domenic, do you think this needs anything else?

Copy link
Contributor

github-actions bot commented Jul 14, 2025

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)

Copy link
Contributor

@domenic domenic left a comment

Choose a reason for hiding this comment

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

Looks great!

@chrisdavidmills chrisdavidmills changed the title Technical review: Add speculationrules request.destination Editorial review: Add speculationrules request.destination Jul 15, 2025
@chrisdavidmills
Copy link
Contributor Author

Technical review done, cheers @domenic.

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

Copy link
Collaborator

@hamishwillee hamishwillee left a comment

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
3 participants