Jericho/ZoomNet

Add `Type` to Recoding model class

Closed this issue · 5 comments

Add `Type` to Recoding model class

From the documentation for API event for recording.completed:

type is integer with following possible values:

If the recording is of a meeting:

1 — Instant meeting.
2 — Scheduled meeting.
3 — A recurring meeting with no fixed time.
4 — A meeting created via PMI (Personal Meeting ID).
7 — A Personal Audio Conference (PAC).
8 - Recurring meeting with a fixed time.

If the recording is of a webinar:

5 — A webinar.
6 — A recurring webinar without a fixed time.
9 — A recurring webinar with a fixed time.

If the recording is not from a meeting or webinar:

99 — A recording uploaded via the Recordings interface on the Zoom Web Portal.

As discussed in a PR recently submitted by @synesthesia, ZoomNet already contains an enum called RecordingType which is used to indicate whether a file is saved locally or in the cloud. Therefore it should have been name something like "RecordingSaveLocation".

More specifically, the existing type is used when configuring meeting (or webinar) settings. When you are configuring these settings, you have the ability to indicate whether the meeting is automatically recorded and if so, where the recording is saved (either in the cloud or local).

Therefore I think RecordingAutoSaveType is a better name. Naming is hard... 😔
Suggestions are welcome!

How about just AutoRecordingType?

🎉 This issue has been resolved in version 0.64.0 🎉

The release is available on:

Your GitReleaseManager bot 📦🚀