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

Add Compute to ParseUri. #927

Merged
merged 1 commit into from Aug 5, 2017

Conversation

Projects
None yet
4 participants
@mtroth-microsoft
Contributor

mtroth-microsoft commented Aug 3, 2017

Need to give some added exposure of the $compute option within the parser by adding it to the ODataUri general parse method.

@mtroth-microsoft mtroth-microsoft requested review from mikepizzo and robward-ms Aug 3, 2017

@msftclas

This comment has been minimized.

msftclas commented Aug 3, 2017

@mtroth-microsoft,
Thanks for your contribution as a Microsoft full-time employee or intern. You do not need to sign a CLA.
Thanks,
Microsoft Pull Request Bot

@@ -166,6 +169,11 @@ public Uri ServiceRoot
public ApplyClause Apply { get; set; }
/// <summary>
/// Gets or sets any $compute option for this uri.
/// </summary>
public ComputeClause Compute { get; set; }

This comment has been minimized.

@robward-ms

robward-ms Aug 4, 2017

Contributor

Can you add a test?

@mtroth-microsoft mtroth-microsoft merged commit 9fd71d9 into OData:master Aug 5, 2017

@mtroth-microsoft mtroth-microsoft deleted the mtroth-microsoft:Feature/ParseUriWithCompute branch Aug 5, 2017

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment