-
Notifications
You must be signed in to change notification settings - Fork 22.8k
docs: Add missing duplex property to RequestInit and update fetch() #40099
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
base: main
Are you sure you want to change the base?
Conversation
Preview URLs Flaws (1)Note! 2 documents with no flaws that don't need to be listed. 🎉 URL:
(comment last updated: 2025-06-28 01:48:25) |
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.
Thank you @Shrinivassab ! I realised we also need a page for the corresponding duplex
property of Request
. You could model this on, for example, https://developer.mozilla.org/en-US/docs/Web/API/Request/redirect.
Accepted PR commented... Co-authored-by: wbamberg <[email protected]>
Accepted PR commented... Co-authored-by: wbamberg <[email protected]>
Accepted PR commented... Co-authored-by: wbamberg <[email protected]>
…tent into add-duplex-docs
Hi @wbamberg I have incorporated your PR comments and in addition to that i added new page for duplex: Please go over it and share your review comments. I will update it based on your suggestions. |
Description
duplex
property toRequestInit
(alphabetically betweencredentials
andheaders
).TypeError
behavior infetch()
exceptions table when:body
is aReadableStream
butduplex
is omitted/not"half"
.duplex
values are provided.Motivation
Additional details
Related issues and pull requests
Fixes #40086