Skip to content
This repository was archived by the owner on Apr 21, 2023. It is now read-only.

Conversation

@iiivanov1993
Copy link

  1. Fix Graph API error by updating the call for creating onlineMeeting object. Currently, "graph.microsoft.com/beta/app/onlineMeetings" with an application token is used to create an onlineMeeting object. It is deprecated as stated in the Graph API documentation: https://docs.microsoft.com/en-us/graph/api/application-post-onlinemeetings?view=graph-rest-beta&tabs=http
    "Note: The /app or /communications path with an application token is deprecated. Going forward, use the /me path with a user token to create online meetings."
    This caused the http request to fail and creation of Meeting inside of Teams returns an error.
    Fixed by using the Graph SDK:
    await GraphClient.Me.OnlineMeetings.Request().AddAsync(onlineMeeting);
  2. Use IWebHostEnvironment instead of deprecated IHostingEnvironment.
  3. Add exception handling pipeline returning exception messages with 500 http responses.

…Use IWebHostEnvironment instead of deprecated IHostingEnvironment. Add exception handling pipeline returning exception messages with 500 http responses.
@msftclas
Copy link

msftclas commented Apr 27, 2020

CLA assistant check
All CLA requirements met.

@jthake jthake requested a review from TBag April 30, 2020 05:20
@jthake
Copy link
Contributor

jthake commented Apr 30, 2020

@TBag Looks like @iiivanov1993 beat us to it from our work tonight on changing to Me.OnlineMeetings. He also has two additional adds I like.

@TBag
Copy link
Collaborator

TBag commented Apr 30, 2020

Let's discuss and decide what to merge today.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants