-
Notifications
You must be signed in to change notification settings - Fork 61
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
DeltaRequestBuilderGetQueryParameters for messages missing option for changeType #364
Comments
This is still not implemented. query_params = DeltaRequestBuilder.DeltaRequestBuilderGetQueryParameters(
changeType = "created"
) fails to it is however clearly documented online: https://learn.microsoft.com/en-us/graph/api/message-delta?view=graph-rest-1.0&tabs=http#query-parameters |
@samwelkanda , @baywet could you please check this? Am I understanding something wrong? I thought that this would get fixed before release 1.0.0, but for some reason it's still not there. :) |
@jussihi thanks for the nudge here. This is caused by this metadata issue. Let me see if I can get it prioritized |
Ah, I see it is some dependency and not tied to the preview-or-not -status of this python library! Thanks for the information. Let's see when we get the feature added! |
Update: the fix to metadata was just merged in today, which means the query parameter should show up in the next generation next week. |
Thanks for the heads-up and quick work on this @baywet ! |
@baywet , which pip module should I upgrade to see this change? I'm still not able to get it working... 😁 |
This is because we haven't released a new version since my last message. Please bear with us while we sort through some organizational/staffing aspects in the background. If this is a pressing matter, please open a support ticket and reference this issue. |
Ok @baywet , thanks for the info. No no there is no hurry, I know it might be a challenging time what we are living in :) I was just wondering if I was doing something wrong here at my end. Thanks for the support again, it is very much appreciated! |
Also https://learn.microsoft.com/en-us/graph/api/calendar-list-calendarview?view=graph-rest-1.0&tabs=python#request (
@baywet , Should I open another issue or is it enough to report this here? |
Also, is there a way to get around this SDK limitation for time being, i.e. can I somehow manually insert these parameters to the request? I would like to get the calendar view one working, since I need it. |
@jussihi This is a different problem (different endpoint/operation) maybe opening a separate issue would have been better here. As for the original issue of the missing change type, it is present in the latest version of the SDK Line 91 in c1cf802
|
Allow to correct the previous statement, it is merged in the main branch, but it seems the package hasn't been published since the original release. https://pypi.org/project/msgraph-sdk/ |
@baywet thank you again for the quick response. Once there is >1.0.0 version available, I will update my python package and report back about these features! 👍 |
This works now! Thank you for support, this issue can be closed :) |
https://learn.microsoft.com/en-us/graph/delta-query-messages#use-query-parameters-in-a-delta-query-for-messages
quoted:
Additionally, to return only certain type of changes (created, updated or deleted) in the delta query's response, you can optionally filter the desired type of change using a custom query option changeType. Possible values are created, updated or deleted.
Regarding my other problems, there doesn't seem to be a way to set the query parameter for delta tracking's "changeType" field. None of this package's
DeltaRequestBuilderGetQueryParameters
has this field. Or should I create my own class inheritingDeltaRequestBuilderGetQueryParameters
and add it there manually?The text was updated successfully, but these errors were encountered: