Skip to content
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

Add meeting link to zoom-notifier #92

Merged
merged 8 commits into from
Feb 28, 2025
Merged

Add meeting link to zoom-notifier #92

merged 8 commits into from
Feb 28, 2025

Conversation

stahnma
Copy link
Owner

@stahnma stahnma commented Feb 28, 2025

No description provided.

This is the first pass with limited functionality of having the application
also be optionally a client to zoom web api and not just a webhook receiver and
processor. Calling the API enables s things like getting the meeting join link
with a passcode or enumerating who is in the meeting.
Previously the join (and leave) message just said "joined the zoom". Now we can
provide a rich text link for others to join. Note, this isn't clean yet. This
commit really puts in the plumbing to call out to the zoom API, not the final
polished experience.
Previously there was no link to the zoom meeting that the zoom-notifier was
sending messages about. This changeset uses rich text formatting via the slack
block builder API to inline a link to the zoom meeting if the proper settings
are provided. These settings are defined in environment variables, but also
require setup on the Zoom application side.

A future commit will have full documentation. This commit is the simple MVP
with a lot of debugging enabled still. This is basically the happy path.
There was a logic error that always assumed ZOOM WEB API was enabled. Now it is
processed properly.
@stahnma stahnma merged commit c7709f1 into main Feb 28, 2025
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant