-
Notifications
You must be signed in to change notification settings - Fork 6
Zoom rules with YAML #1078
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
base: rc-v0.5.18
Are you sure you want to change the base?
Zoom rules with YAML #1078
Changes from all commits
0ca4bbe
435a3fe
4f1f9a1
09d846d
3db1ba1
1cee218
83e6fc4
f3d556f
e4437dc
6b2873a
0ac4b59
4e25621
5a6db2d
6578689
fa56173
d14dd4e
e24af32
4597fac
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
|
Contributor
Author
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Not used. Files, test and endpoint dropped. |
This file was deleted.
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -5,7 +5,6 @@ | |
| "total_records":4, | ||
| "meetings":[ | ||
| { | ||
| "uuid":"mlghmfghlBBB", | ||
|
Contributor
Author
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Same: not used and not present https://developers.zoom.us/docs/api/meetings/#tag/meetings/get/users/{userId}/meetings |
||
| "id":11111, | ||
| "host_id":"t~YCq-iwWo1b500r_AH_4tKLuP0oBLrlwoHEpY7DSxi8c", | ||
| "type":2, | ||
|
|
@@ -15,7 +14,6 @@ | |
| "created_at":"2019-08-16T01:13:12Z" | ||
| }, | ||
| { | ||
| "uuid":"J8H8eavweUcd321==", | ||
| "id":2222, | ||
| "host_id":"t~YCq-iwWo1b500r_AH_4tKLuP0oBLrlwoHEpY7DSxi8c", | ||
| "type":2, | ||
|
|
@@ -25,7 +23,6 @@ | |
| "created_at":"2019-08-16T18:30:46Z" | ||
| }, | ||
| { | ||
| "uuid":"SGVTAcfSfCbbbb", | ||
| "id":33333, | ||
| "host_id":"t~YCq-iwWo1b500r_AH_4tKLuP0oBLrlwoHEpY7DSxi8c", | ||
| "type":2, | ||
|
|
@@ -35,7 +32,6 @@ | |
| "created_at":"2019-08-16T21:15:56Z" | ||
| }, | ||
| { | ||
| "uuid":"64123avdfsMVA==", | ||
| "id":44444, | ||
| "host_id":"t~YCq-iwWo1b500r_AH_4tKLuP0oBLrlwoHEpY7DSxi8c", | ||
| "type":2, | ||
|
|
@@ -45,4 +41,4 @@ | |
| "created_at":"2019-08-29T17:32:33Z" | ||
| } | ||
| ] | ||
| } | ||
| } | ||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
These
pmiare not present in the responde model from API and not used in our source model. See https://developers.zoom.us/docs/api/meetings/#tag/meetings/get/past_meetings/{meetingId}/participantsThere was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ok, yeah - either this was present in the example json at some point, or we put it in there just to be SURE would be redacted if ever appeared in some zoom versions.
but the responseSchema approach achieves that now.