Skip to content

Conversation

@andrueastman
Copy link
Contributor

@andrueastman andrueastman commented Feb 23, 2022

This PR syncs relevant changes on the dev branch with the feature/3.0 branch,

Changes include:

Microsoft Reviewers: Open in CodeFlow

Andrew Omondi added 4 commits February 23, 2022 12:59
…Bugs

Fix for BatchResponseContent.GetResponseByIdAsync nullref and incorrect content header for non json response
Always ConfiguteAwait false on awaited tasks
@andrueastman andrueastman self-assigned this Feb 23, 2022
@andrueastman andrueastman added the Kiota Is solved by Kiota or will be addressed in Kiota label Feb 23, 2022
@andrueastman andrueastman added this to the Kiota milestone Feb 23, 2022
@andrueastman andrueastman marked this pull request as ready for review February 23, 2022 10:28
@andrueastman andrueastman requested review from baywet and removed request for MIchaelMainer, peombwa and zengin February 23, 2022 10:28
Copy link
Member

@baywet baywet left a comment

Choose a reason for hiding this comment

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

the configure await changes might need to get reflected in the kiota libs as well.

/// Gets the <see cref="IRequestAdapter"/> for sending requests.
/// </summary>
protected internal IRequestAdapter RequestAdapter { get; set; }
protected internal virtual IRequestAdapter RequestAdapter { get; set; }
Copy link
Member

Choose a reason for hiding this comment

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

is the only point of this class to expose the batch property to the fluent API?

@andrueastman andrueastman merged commit 5620fe6 into feature/3.0 Feb 23, 2022
@andrueastman andrueastman deleted the andrueastman/syncDevChanges branch February 23, 2022 13:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Kiota Is solved by Kiota or will be addressed in Kiota

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants