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

Update aad-advanced-queries.md #9468

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
1 change: 1 addition & 0 deletions concepts/aad-advanced-queries.md
Original file line number Diff line number Diff line change
Expand Up @@ -152,6 +152,7 @@ The following table lists query scenarios on directory objects that are supporte
> + `$expand` is not currently supported with advanced queries.
> + The advanced query capabilities are currently not available for Azure AD B2C tenants.
> + To use advanced query capabilities in [batch requests](json-batching.md), specify the **ConsistencyLevel** header in the JSON body of the `POST` request.
> + Using a mix of native and dynamic files is not supported. For example: /drives/{{ContainerID}}/items?$filter=listitem/fields/{{columnName}} eq '{{ColumnValue}}' AND name eq '{{DocumentName}}' should be formatted as: /drives/{{ContainerID}}/items?$filter=listitem/fields/{{columnName}} eq '{{ColumnValue}}' AND listitem/fields/FileLeafRef eq '{{DocumentName}}'
Copy link
Contributor

Choose a reason for hiding this comment

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

This article is specific to Microsoft Entra, but you are adding details about a specific relationship on drive resources. There currently is no page for the items relationship on drive so that will first need to be created and then this information should go in there.

Copy link
Contributor

Choose a reason for hiding this comment

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

Or I don't really know if relationships get their own docs page. We should figure out how to document this, but a Microsoft Entra page is not the right place.

Copy link
Contributor

Choose a reason for hiding this comment

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

It could be that we just need to give a consistent behaviour for all driveItem collections (so OData support should probably be consistent on GET drive/items, drive/special, drive/following, drive/bundles, driveItem/children, etc.

Copy link
Contributor

Choose a reason for hiding this comment

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

Per Diego's comment, this is not the right topic for this information. Would it make sense to add it to the overview topic for the Files node? https://learn.microsoft.com/en-us/graph/api/resources/onedrive?view=graph-rest-1.0

Copy link
Author

Choose a reason for hiding this comment

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

@dluces considering that this functionality is specific to drive items, I suggest we expand the get driveItem to include standard filter functionality. We should also include the limitations were seeing on children on the List children page.

Copy link
Contributor

Choose a reason for hiding this comment

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

@Spucelik the "get driveItem" page is to get a single driveItem (so when you have a driveItem id). There's no pagination or filtering when you get a single object. We just don't have a page yet to get driveItems within a drive which is what this applies to.

By the way, the "List children" page has a completely different set of limitations than the "drive.items" one.


## Support for filter by properties of Microsoft Entra ID (directory) objects

Expand Down