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 group-list.md #9437

Merged
merged 4 commits into from
Oct 15, 2024
Merged

Update group-list.md #9437

merged 4 commits into from
Oct 15, 2024

Conversation

CMMOliveira
Copy link
Contributor

Executing this example 2 with "ConsistencyLevel - eventual" it fails.

Bad Request - 400
"code": "Request_UnsupportedQuery",
"message": "Unsupported or invalid query filter clause specified for property 'hasMembersWithLicenseErrors' of resource 'Group'.",

It will work without the ConsistencyLevel.

Executing this example with "ConsistencyLevel - eventual" it fails.


Bad Request - 400 
        "code": "Request_UnsupportedQuery",
        "message": "Unsupported or invalid query filter clause specified for property 'hasMembersWithLicenseErrors' of resource 'Group'.",
Copy link

Learn Build status updates of commit d70b2d5:

❌ Validation status: errors

Please follow instructions here which may help to resolve issue.

File Status Preview URL Details
api-reference/beta/api/educationalactivity-delete.md ❌Error Details
api-reference/beta/api/educationalactivity-get.md ❌Error Details
api-reference/beta/api/educationalactivity-update.md ❌Error Details

api-reference/beta/api/educationalactivity-delete.md

  • Line 37, Column 1: [Error: include-not-found - See documentation] Invalid include link: '../includes/permissions/educationalactivity-delete-permissions.md'.

api-reference/beta/api/educationalactivity-get.md

  • Line 37, Column 1: [Error: include-not-found - See documentation] Invalid include link: '../includes/permissions/educationalactivity-get-permissions.md'.

api-reference/beta/api/educationalactivity-update.md

  • Line 37, Column 1: [Error: include-not-found - See documentation] Invalid include link: '../includes/permissions/educationalactivity-update-permissions.md'.

For more details, please refer to the build report.

Note: Your PR may contain errors or warnings or suggestions unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.

For any questions, please:

@Lauragra Lauragra added the review requested Use for external PRs that need to be reviewed by content author. label Oct 10, 2024
@Lauragra
Copy link
Contributor

@yuhko-msft , @FaithOmbongi , can you verify this PR please? Should the header work when the $count parameter is used?

Copy link

Learn Build status updates of commit 0558258:

✅ Validation status: passed

File Status Preview URL Details
api-reference/v1.0/api/group-list.md ✅Succeeded

For more details, please refer to the build report.

For any questions, please:

@FaithOmbongi FaithOmbongi self-assigned this Oct 14, 2024
@FaithOmbongi FaithOmbongi added area: Entra APIs and issues relating to Microsoft Entra (previously Azure AD) features. area: groups content review in progress and removed review requested Use for external PRs that need to be reviewed by content author. labels Oct 14, 2024
Copy link
Collaborator

@FaithOmbongi FaithOmbongi left a comment

Choose a reason for hiding this comment

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

Thank you @CMMOliveira for proposing these changes. The property is one in a list of properties that doesn't soft-fail if you use advanced query parameters. I'll raise a separate update to update the information here https://learn.microsoft.com/en-us/graph/aad-advanced-queries?tabs=http

Copy link

Learn Build status updates of commit 3e7acad:

✅ Validation status: passed

File Status Preview URL Details
api-reference/beta/api/group-list.md ✅Succeeded
api-reference/v1.0/api/group-list.md ✅Succeeded

For more details, please refer to the build report.

For any questions, please:

@FaithOmbongi FaithOmbongi added content review complete Apply when the content review is complete on a PR. ready to merge Use this label to indicate to the repository admins that your PR is ready to merge into master. and removed content review in progress labels Oct 14, 2024
@Lauragra Lauragra merged commit f8022b8 into microsoftgraph:main Oct 15, 2024
6 of 7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area: Entra APIs and issues relating to Microsoft Entra (previously Azure AD) features. area: groups content review complete Apply when the content review is complete on a PR. ready to merge Use this label to indicate to the repository admins that your PR is ready to merge into master.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants