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

Add Support to specifiy ODataQuery Parameters in Microsoft Graph Client #827

Conversation

pri-kise
Copy link
Contributor

@pri-kise pri-kise commented Mar 25, 2024

Summary

This PR adds support so specify OData Query Parameters via the Microsoft Graph Client

Work Item(s)

Fixes #802

Fixes AB#522447

Fixes AB#527604

@pri-kise pri-kise requested a review from a team as a code owner March 25, 2024 16:17
@github-actions github-actions bot added AL: System Application From Fork Pull request is coming from a fork labels Mar 25, 2024
@github-actions github-actions bot added this to the Version 25.0 milestone Apr 3, 2024
@JesperSchulz JesperSchulz added the Integration GitHub request for Integration area label Apr 22, 2024
AndreasMoth
AndreasMoth previously approved these changes Apr 26, 2024
@JesperSchulz JesperSchulz added the Linked Issue is linked to a Azure Boards work item label Apr 26, 2024
JesperSchulz
JesperSchulz previously approved these changes Apr 26, 2024
Copy link
Contributor

@JesperSchulz JesperSchulz left a comment

Choose a reason for hiding this comment

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

Let us get this one on the road! 🥳

@JesperSchulz JesperSchulz dismissed stale reviews from AndreasMoth and themself via 9526963 April 26, 2024 11:31
@JesperSchulz
Copy link
Contributor

@pri-kise, once you've fixed the remaining codecop issues, we're ready to merge. Seems like you need to order some variables in your test and that maybe a namespace is missing...

@JesperSchulz JesperSchulz self-assigned this Apr 30, 2024
@JesperSchulz JesperSchulz merged commit 5e67602 into microsoft:main May 1, 2024
23 checks passed
@pri-kise
Copy link
Contributor Author

pri-kise commented May 2, 2024

@JesperSchulz can this maybe be backported to 24.x ?

@JesperSchulz
Copy link
Contributor

@JesperSchulz can this maybe be backported to 24.x ?

Of course. I'll create a PR.

@JesperSchulz
Copy link
Contributor

PR created: #1042

JesperSchulz added a commit that referenced this pull request May 7, 2024
…ph Client (#1042)

#### Summary
Backport of #827 

#### Work Item(s)
Fixes #802
Fixes
[AB#533480](https://dynamicssmb2.visualstudio.com/1fcb79e7-ab07-432a-a3c6-6cf5a88ba4a5/_workitems/edit/533480)

---------

Co-authored-by: pri-kise <56249171+pri-kise@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
AL: System Application From Fork Pull request is coming from a fork Integration GitHub request for Integration area Linked Issue is linked to a Azure Boards work item
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[BC Idea]: Improve Microsoft Graph Module with OData Query Parameters
4 participants