Skip to content
This repository was archived by the owner on Jan 5, 2026. It is now read-only.

feat: Include HEAD method to http requests - #6713

Merged
tracyboehrer (tracyboehrer) merged 1 commit into
mainfrom
southworks/add/httprequest-head-method
Dec 14, 2023
Merged

tracyboehrer (tracyboehrer) merged 1 commit into
mainfrom
southworks/add/httprequest-head-method

Conversation

@JhonatanSVDev

Copy link
Copy Markdown
Collaborator

#minor

Description

This PR adds the HEAD method in the HttpRequest class and adds the method in every schema

Specific Changes

  • Added HEAD method to the HttpRequest method list.
  • Added HEAD method to schemas of testing and HttpRequest.

Testing

The following image shows the dialogs unit tests passing after the changes.
image

@JhonatanSVDev Jhonatan Sandoval Velasco (JhonatanSVDev) added the Automation: No parity PR does not need to be applied to other languages. label Dec 12, 2023
@coveralls

Copy link
Copy Markdown
Collaborator

Pull Request Test Coverage Report for Build 380839

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • 9 unchanged lines in 4 files lost coverage.
  • Overall coverage increased (+0.009%) to 78.458%

Files with Coverage Reduction New Missed Lines %
/libraries/AdaptiveExpressions/BuiltinFunctions/GetNextViableTime.cs 1 90.91%
/libraries/AdaptiveExpressions/BuiltinFunctions/GetPreviousViableTime.cs 1 90.91%
/libraries/integration/Microsoft.Bot.Builder.Integration.AspNet.Core/ServiceCollectionExtensions.cs 1 93.55%
/libraries/Microsoft.Bot.Builder.Dialogs.Adaptive/Actions/HttpRequest.cs 6 87.93%
Totals Coverage Status
Change from base Build 380329: 0.009%
Covered Lines: 26135
Relevant Lines: 33311

💛 - Coveralls

@tracyboehrer
tracyboehrer (tracyboehrer) deleted the southworks/add/httprequest-head-method branch December 14, 2023 14:50
This was referenced May 14, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Automation: No parity PR does not need to be applied to other languages.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants