-
Notifications
You must be signed in to change notification settings - Fork 196
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
Enrich metadata model to describe supported functionalities #38
Comments
$select/$expand typically works (we could tap into https://github.com/microsoftgraph/msgraph-metadata). $filter and $orderby are more complex ones as some fields work with $filter, some don't. Required permissions can be fetched from https://github.com/microsoftgraph/msgraph-metadata as well |
Some additional thoughts:
|
TestQueryTeamChannelMessages fails due to $top being added while it's not supported |
Closing this one: a ton of work and the added value will be just cleaner error messages in case of a mission permission or wrongly used query parameter. Also checking on query parameters like $filter is not so obvious, in graph one can filter on some fields but not on all...there's no easy way to understand which fields are filterable and having a check in PnP Core SDK would then mean we always have to keep maintaining this metadata. |
Make it possible to mark domain model types, properties, and method with requirements (from a security point of view: delegated, app-only, both) and with OData support (like: $select, $filter, $orderby, $top, $skip, $expand, etc.).
The text was updated successfully, but these errors were encountered: