Repository navigation
Zoom Recaps to Slack: only post recaps from the last 48 hours - #7
Closed
rrrutledge wants to merge 2 commits into
Closed
rrrutledge wants to merge 2 commits into
rrrutledge wants to merge 2 commits into
Conversation
…k handles only its own meeting Zoom's summaries list ignores the date window and returns meetings back to 2024, so a run with nothing new dug into history and posted old recaps almost daily. Now a meeting that started more than 48 hours ago is never a candidate, and a webhook run considers only the exact occurrence that fired it, with no fallback to another occurrence of the series. Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01AEavKGR3wFFUQXR9W16mSQ
…ff, rewrap header Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01AEavKGR3wFFUQXR9W16mSQ
rrrutledge
marked this pull request as ready for review
November 7, 2026 21:34
Contributor
Author
|
Superseded by #8, which fixes the same old-recap bug on main. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Zoom Recaps to Slack has been posting one old recap almost every day since 10-29 (Summit Planning from 10-19, Board from 04-23 and 08-27, and so on), which Addie noticed on 11-05.
Zoom's summaries list ignores the date window and returns about 994 summaries back to June 2024.
Build Candidates picked the newest unposted occurrence from any date, so any run with nothing new dug into history.
A webhook whose own meeting the 8:00 AM sweep had already posted fell back to the series' newest unposted occurrence.
Now a meeting that started more than 48 hours ago is never a candidate, for both the sweep and the webhook.
A webhook run considers only the exact occurrence that fired it; if that one is already posted or not ready, the run ends with nothing.
Review findings rejected:
The 1-hour wait before posting is already in
main(#3), so no other change is needed here.🤖 Generated with Claude Code