-
Notifications
You must be signed in to change notification settings - Fork 474
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
OData60 context url Fixes #948
base: OData60
Are you sure you want to change the base?
Conversation
…ined nav props and functions.
#endregion FunctionImports | ||
|
||
|
||
private IQueryable<Appointment> getAppointments() |
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.
GetAppointments() is better. :)
return Ok(getAppointments()); | ||
} | ||
|
||
//[HttpGet] |
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.
remove the comment cases.
|
||
[HttpGet] | ||
[ODataRoute("Businesses({BusinessId})/Default.boundNavPath()")] | ||
public IHttpActionResult boundNavPath([FromODataUri]int BusinessId) |
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.
lower case for parameter
} | ||
|
||
//[HttpGet] | ||
//[EnableQuery] |
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.
remove comment cases or add the comments
using Nuwa; | ||
using Microsoft.OData.Edm; | ||
using System.ComponentModel.DataAnnotations; | ||
using System.Runtime.CompilerServices; |
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.
sort
@mikepizzo any update for this? |
Issues
This pull request fixes issue #917.
Description
Passes Operation Path down to ODL so that it can correctly generate a ContextUrl.
Relies on PR776 from OData.net (OData/odata.net#776)
Checklist (Uncheck if it is not completed)
Additional work necessary
No Docs Needed.
*Commented test code validates appending key lookup to function invocation which is apparently not supported in WebAPI OData today.