-
Notifications
You must be signed in to change notification settings - Fork 1
Automatically scheduling zoom talks and sending keys #16
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
Conversation
|
BTW, here the code for talking with zoom can be copies almost completely from #5 |
|
Let's separate issues, and implement host key rotation separately (I'm looking into that). We'll arrange the sending at a later stage. |
Also if the meeting is configured to send an email upon registration, then participants registered via API will receive a registration confirmation, resulting in the speaker getting 2 emails: one with the host key, and another one with their join URL. As a workaround we can create the meeting with Alternatively we can register the speaker with a wrong email. That's faster, but less classy. |
|
@everthemore I pushed the implementation of host key rotation to the repo. |
|
cron runs aren't on yet, but I checked that key updates work. |
|
@everthemore the keys are fully in place, and should be easy to reuse. Let's get this one going now. |
|
I also put some reusable zoom functionality into |
|
@akhmerov I added the 'add talk to researchseminars.org' file here too, since it makes sense to do so as soon as the zoom link has been registered. The code is WIP, meaning that most likely some of the YAML fields are incorrectly named atm. I will fix this before we start testing of course. There are a few places in the code that I would like input on, I'll highlight them in review. |
|
Closes #20 |
|
I have a temporary bit in there that prevents updating of the yaml file with talks, for the first few tests |
|
Plan:
|
What did you have in mind here? Should this instead be done by the host-key rotator? |
No, that one runs on schedule and doesn't know whom to contact. We should of course reuse the function that generates the host key for a given time slot, but call if from this workflow. |
|
Use mailgun? |
|
@akhmerov Zoom link for meeting is missing, and I'm sure other things are not up to standards. Hopefully I can tweak from here. |
|
Like adding the time slot to the email |
|
@akhmerov Merge for testing? |
|
My linter catches a bunch of things, let me fix those first. |
|
I propose to disable research seminars registration at first: it's easier to debug one thing at a time. Plus authorization is missing in one of the function signatures. Other than that I think good to merge and debug. |
No description provided.