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.

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

Exposed underlying CosmosElements on item QueryResponse<T> using gene… #1018

Merged
merged 4 commits into from
Nov 20, 2019

Conversation

john-pao
Copy link
Contributor

Pull Request Template

Description

Exposed underlying CosmosElements on item QueryResponse using generic type parameter.

Type of change

  • New feature (non-breaking change which adds functionality)

j82w
j82w previously approved these changes Nov 18, 2019
- Switching to using 'as' rather than 'is' to save on reflection.
bchong95
bchong95 previously approved these changes Nov 18, 2019
Copy link
Contributor

@bchong95 bchong95 left a comment

Choose a reason for hiding this comment

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

Thanks for making this change :)

- Changing 'as' and 'null' check to inline 'is'.
bchong95
bchong95 previously approved these changes Nov 19, 2019
- Trying to do the generic cast in all cases leads to CosmosElements being returned for QueryResponse<dynamic>.Resource and QueryResponse<object>.Resource, etc.
- This gives us the least change to the behavior for users that might do these sorts of casts.
Copy link
Contributor

@sboshra sboshra left a comment

Choose a reason for hiding this comment

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

:shipit:

@sboshra sboshra merged commit 7ae5c61 into master Nov 20, 2019
@sboshra sboshra deleted the users/jopao/queryResponse branch November 20, 2019 19:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants