-
Notifications
You must be signed in to change notification settings - Fork 1.6k
missing doc for HttpRequestOptions #7129
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
Conversation
…uestMessage.Options
Tagging subscribers to this area: @dotnet/ncl Issue Detailsmissing doc for HttpRequestOptions, HttpRequestOptionsKey and HttpRequestMessage.Options
|
Docs Build status updates of commit 908a206: ✅ Validation status: passed
For more details, please refer to the build report. Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report. For any questions, please:
|
@carlossanlop @gewarren The generated preview for HttpRequestMessage shows obsolete |
The preview link always defaults to the UWP moniker, for some reason. How does this look? https://review.docs.microsoft.com/en-us/dotnet/api/system.net.http.httprequestmessage?view=net-6.0&branch=pr-en-us-7129 |
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.
I left some suggestions.
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.
Thanks for this change, @pavelsavara . Please merge after applying the suggestions provided by @gewarren.
Co-authored-by: Genevieve Warren <24882762+gewarren@users.noreply.github.com>
Docs Build status updates of commit a265def: ✅ Validation status: passed
For more details, please refer to the build report. Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report. For any questions, please:
|
missing doc for HttpRequestOptions, HttpRequestOptionsKey and HttpRequestMessage.Options
Contributes to dotnet/runtime#43859
After this is merged, relevant runtime change is dotnet/runtime#58896