Skip to content

Fixed filter query - #9996

Open
MartinM85 wants to merge 1 commit into
microsoftgraph:mainfrom
MartinM85:fix/user-list-sponsorof-filter-queries
Open

Fixed filter query#9996
MartinM85 wants to merge 1 commit into
microsoftgraph:mainfrom
MartinM85:fix/user-list-sponsorof-filter-queries

Conversation

@MartinM85

Copy link
Copy Markdown
Contributor

Important

Use the AI-powered workflow for Graph onboarding to author and self-review schema-related changes. See https://aka.ms/msgraphcdk for more information.

Required for API changes:

  • [] Link to API.md file: ADD LINK HERE
  • [] Link to PR for public-facing schema changes (schema-Prod-beta/v1.0.csdl): ADD LINK HERE

Add other supporting information, such as a description of the PR changes:

This PR fixes filter query which is missing the $count query parameter. Without the $count=true, the request fails with the error Request_UnsupportedQuery and message The specified filter to the reference property query is currently not supported. It's an advance query.


Important

The following guidance is for Microsoft employees only. Community contributors can ignore this message; our content team will manage the status.

After you've created your PR, expand this section for tips and additional instructions.
  • do not merge is the default PR status and is automatically added to all open PRs that don't have the ready to merge label.
  • Add the ready for content review label to start a review. Only PRs that have met the minimum requirements for content review and have this label are reviewed.
  • If your content reviewer requests changes, review the feedback and address accordingly as soon as possible to keep your pull request moving forward. After you address the feedback, remove the changes requested label, add the review feedback addressed label, and select the Re-request review icon next to the content reviewer's alias. If you can't add labels, add a comment with #feedback-addressed to the pull request.
  • After the content review is complete, your reviewer will add the content review complete label. When the updates in this PR are ready for external customers to use, replace the do not merge label with ready to merge and the PR will be merged within 24 working hours.
  • Pull requests that are inactive for more than 6 weeks will be automatically closed. Before that, you receive reminders at 2 weeks, 4 weeks, and 6 weeks. If you still need the PR, you can reopen or recreate the request.

For more information, see the Content review process summary.

@learn-build-service-prod

Copy link
Copy Markdown
Contributor

Learn Build status updates of commit 3352055:

✅ Validation status: passed

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

For more details, please refer to the build report.

@v-regandowner

Copy link
Copy Markdown

@Jackson-Woods

Can you review the proposed changes?

IMPORTANT: When the changes are ready for publication, adding a #sign-off comment and the "ready-to-merge" label is the best way to signal that the PR is ready for the review team to merge.

@v-regandowner v-regandowner added the aq-pr-triaged PR has been processed by the Aquent PR review team label Jul 24, 2026
@Danipocket Danipocket added the blocked PRs that are blocked from content review or getting merged for some reason. label Jul 27, 2026
@Danipocket

Copy link
Copy Markdown
Contributor

hi! We can't review or approve PRs with major changes through this repo. Please direct internal contributors to submit their PRs through our private repo instead. External contributors can propose minor or cosmetic changes and for major changes, they should contact each workload directly.

@MartinM85

MartinM85 commented Jul 28, 2026

Copy link
Copy Markdown
Contributor Author

Hi @Danipocket, what exactly are minor or cosmetic changes? Why can't I deliver changes that fixes broken examples? How can I contact the workload directly? Are there any exceptions for MVPs to deliver more than cosmetic changes?

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

Labels

aq-pr-triaged PR has been processed by the Aquent PR review team blocked PRs that are blocked from content review or getting merged for some reason.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants