Skip to content

feat: Zoom meeting webhooks and user timezone support#775

Merged
bokelley merged 3 commits into
mainfrom
bokelley/test-zoom-calendar
Jan 15, 2026
Merged

feat: Zoom meeting webhooks and user timezone support#775
bokelley merged 3 commits into
mainfrom
bokelley/test-zoom-calendar

Conversation

@bokelley

Copy link
Copy Markdown
Contributor

Summary

  • Handle Zoom meeting.started and meeting.ended webhooks to update meeting status in real-time
  • Handle recording.completed webhook to store transcripts and fetch Zoom AI Companion meeting summaries
  • Send Slack notifications to working group channels when meetings start/end
  • Fix JSON body parsing middleware to enable Zoom webhook signature verification
  • Add user timezone column with migration for scheduling preferences

Test plan

  • Start a Zoom meeting that's tracked in the database and verify meeting.started webhook updates status
  • Verify Slack notification is sent when meeting starts (if working group has Slack channel configured)
  • End the meeting and verify meeting.ended webhook updates status
  • Verify Slack notification is sent when meeting ends with duration
  • Complete a recording and verify transcript is stored
  • Verify Zoom AI Companion summary is fetched and stored (if available)
  • Test timezone migration runs successfully

🤖 Generated with Claude Code

bokelley and others added 3 commits January 15, 2026 07:31
- Handle meeting.started/ended webhooks to update meeting status
- Handle recording.completed to store transcripts and Zoom AI summaries
- Send Slack notifications when meetings start/end
- Add user timezone column with migration
- Add helper functions for timezone management

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Migration 170 already exists in main (slack_activity_index).

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Server implementation changes (webhooks, migrations) don't require
a protocol version bump.

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
@bokelley bokelley merged commit 422fe9b into main Jan 15, 2026
5 checks passed
bokelley added a commit that referenced this pull request Jan 16, 2026
* feat: Add Zoom meeting webhooks and user timezone support

- Handle meeting.started/ended webhooks to update meeting status
- Handle recording.completed to store transcripts and Zoom AI summaries
- Send Slack notifications when meetings start/end
- Add user timezone column with migration
- Add helper functions for timezone management

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>

* fix: Rename user_timezone migration to 171 to avoid conflict

Migration 170 already exists in main (slack_activity_index).

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>

* fix: Use empty changeset for server-only changes

Server implementation changes (webhooks, migrations) don't require
a protocol version bump.

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>

---------

Co-authored-by: Claude Opus 4.5 <noreply@anthropic.com>
bokelley added a commit that referenced this pull request Jan 20, 2026
* feat: Add Zoom meeting webhooks and user timezone support

- Handle meeting.started/ended webhooks to update meeting status
- Handle recording.completed to store transcripts and Zoom AI summaries
- Send Slack notifications when meetings start/end
- Add user timezone column with migration
- Add helper functions for timezone management

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>

* fix: Rename user_timezone migration to 171 to avoid conflict

Migration 170 already exists in main (slack_activity_index).

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>

* fix: Use empty changeset for server-only changes

Server implementation changes (webhooks, migrations) don't require
a protocol version bump.

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>

---------

Co-authored-by: Claude Opus 4.5 <noreply@anthropic.com>
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