virtualscienceforum / automation

Scripts for running the VSF
BSD 2-Clause "Simplified" License
0 stars 1 forks source link

Custom zoom registration form #25

Closed everthemore closed 3 years ago

everthemore commented 4 years ago

Update cloudflare worker to also be able to handle zoom registrations. TODO:

Closes #15

akhmerov commented 4 years ago

The plan sounds good. If the meeting is running, the client side should also immediately open the join URL (adding this for later).

everthemore commented 3 years ago

If you get a chance @akhmerov, take a look at this workflow ('wranger dev' from top directory will start a local version to test).

everthemore commented 3 years ago

TODO: Add boolean flag for Zoom yo notify registrant rather than custom email

akhmerov commented 3 years ago

In order to minimize disruption of the mailing list signups, let's check whether we can deploy 2 workers.

everthemore commented 3 years ago

[ ] For simplicity let's use the default zoom registration emails

The registration via the API has the auto_approve flag that we can test.

akhmerov commented 3 years ago

The registration via the API has the auto_approve flag that we can test.

I remembered wrong. Whether an email is sent or not, is actually meeting configuration, and it's already correctly set to true. So this requires no action.

everthemore commented 3 years ago

[x] Important immediately after merging we should test whether mailing list signups still work.

Can confirm that this still works. The cloudflare worker now has a separate endpoint for mailing list signups, and one for zoom registrations that we can test separately by creating mock talks.

akhmerov commented 3 years ago

I propose:

  1. Merge this
  2. Immediately push an update pointing the mailing list subscription to the new URL (separately from the other form updates)
  3. Test whether mailing list subscription still works
  4. Proceed with other improvements without time pressure.