Skip to content
This repository has been archived by the owner on Mar 10, 2023. It is now read-only.

Updated the proposal to reflect the landing of changes in fetch #72

Merged
merged 1 commit into from
Dec 26, 2022

Conversation

pmeenan
Copy link
Collaborator

@pmeenan pmeenan commented Dec 16, 2022

The fetch spec has merged the changes so that part of the IDL is now duplicate here and needed to be removed to prevent Web Platform Test issues.

The proposed changes to fetch section was also no longer relevant and has been updated to point to fetch.

I also updated the language in the explainer paragraphs to reflect the current state of the spec in that it is no longer "will" but has shipped for the browsers that support it and isn't expected to change.

Once the changes land in HTML, that section will also be removed and this can be reduced to a document that points to the relevant specs and provides the additional info on use cases (should probably not be deleted entirely so inbound links don't just break).

Copy link
Contributor

@yoavweiss yoavweiss left a comment

Choose a reason for hiding this comment

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

LGTM

@yoavweiss
Copy link
Contributor

Once the changes land in HTML, that section will also be removed and this can be reduced to a document that points to the relevant specs and provides the additional info on use cases (should probably not be deleted entirely so inbound links don't just break).

Typically, we redirect such specifications to their new locations. Given that this will be merged to multiple specifications, it might be best to redirect the old specification to the explainer and point to the relevant current specifications from there.

@pmeenan pmeenan merged commit fc49ea4 into WICG:main Dec 26, 2022
github-actions bot added a commit that referenced this pull request Dec 26, 2022
SHA: fc49ea4
Reason: push, by pmeenan

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
github-actions bot added a commit to pmeenan/priority-hints that referenced this pull request Jan 10, 2023
SHA: fc49ea4
Reason: push, by pmeenan

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants