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.

Sign up for GitHub

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

Jump to bottom

Editorial: Use "process the next redirect" instead of "Navigation redirect" #8097

Merged
merged 3 commits into from
Oct 17, 2022
Merged

Editorial: Use "process the next redirect" instead of "Navigation redirect" #8097

merged 3 commits into from
Oct 17, 2022

Conversation

noamr
Copy link
Contributor

@noamr noamr commented Jul 11, 2022 •
edited by pr-preview bot
Loading

This lets FETCH keep the existing request/response/fetch-params.

See whatwg/fetch#1469

/browsing-the-web.html ( diff )
/infrastructure.html ( diff )

@noamr
Use "process the next redirect" instead of "Navigation redirect"
f8f263d 
This lets FETCH keep the existing request/response/fetch-params.
@noamr noamr mentioned this pull request Jul 11, 2022
Closed
3 tasks
domenic
domenic approved these changes Jul 12, 2022
View reviewed changes
Copy link
Member

@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.

LGTM with nit

source Show resolved Hide resolved
@noamr noamr mentioned this pull request Sep 2, 2022
Merged
3 tasks
@noamr noamr closed this Oct 14, 2022
@noamr noamr reopened this Oct 14, 2022
@domenic domenic merged commit 2e8e916 into whatwg:main Oct 17, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@domenic domenic domenic approved these changes

Assignees
No one assigned
Labels
topic: fetch topic: navigation
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@noamr @domenic