Jericho / ZoomNet

.NET client library for the Zoom.us REST API v2
MIT License
68 stars 48 forks source link

Mettings method - GetAsync #85

Closed klides closed 3 years ago

klides commented 3 years ago

Hi, The GetAsync method on metting you are using Meeting class to serialize. That class doesn´t has duration, startTime and timezone. Can you do that with ScheduledMeeting class? Same on GetAll method.

Regards!

Jericho commented 3 years ago

The Meeting class is an abstract base class for InstantMeeting, ScheduledMeeting and RecurringMeeting. The GetAsync method takes care of converting the JSON payload returned by the Zoom API into one of the three concrete classes which derive from the Meeting base class.

In other words, if you are retrieving a scheduled meeting you can write somethin like this:

var scheduledMeeting = (ScheduledMeeting)await client.Meetings.GetAsync(myMeetingId, null, cancellationToken).ConfigureAwait(false);
await log.WriteLineAsync($"Scheduled meeting {myMeetingId} retrieved").ConfigureAwait(false);
klides commented 3 years ago

Thank you so much for your fast reply!

Tell me onde thing, my project is on Asp.Net Core 2.1 now. It says that supports ASP.NET Core 2.x but the release has Microsoft.Extensions.Logging 3.14... Any idea how I can make it run with Microsoft.Extensions.Logging 2.x?

Best regards!

Em qui., 21 de jan. de 2021 às 15:50, Jericho notifications@github.com escreveu:

The Meeting class is an abstract base class for InstantMeeting, ScheduledMeeting and RecurringMeeting. The GetAsync method takes care of converting the JSON payload returned by the Zoom API into one of the three concrete classes which derive from the Meeting base class.

In other words, if you are retrieving a scheduled meeting you can write somethin like this:

var scheduledMeeting = (ScheduledMeeting)await client.Meetings.GetAsync(myMeetingId, null, cancellationToken).ConfigureAwait(false);await log.WriteLineAsync($"Scheduled meeting {myMeetingId} retrieved").ConfigureAwait(false);

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/Jericho/ZoomNet/issues/85#issuecomment-764737527, or unsubscribe https://github.com/notifications/unsubscribe-auth/ACA6IFBWPXXDRHUJHNL4663S3BEMPANCNFSM4WM67PSA .

-- Paulo Fernandes

Jericho commented 3 years ago

As far as I know, Microsoft.Extensions.Logging should be compatible with Asp.Net Core 2.1, is that not the case?

Reviewing this package on NuGet, I see that it targets netframework461, netstandard2.0, netstandard2.1 and net5.0.