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.

Sign up for GitHub

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

Jump to bottom

Populate calendar with only items that were not deleted #645

Merged
merged 2 commits into from
Mar 6, 2025
Merged

Populate calendar with only items that were not deleted #645

merged 2 commits into from
Mar 6, 2025

Conversation

cbounphengsy
Copy link
Contributor

@cbounphengsy cbounphengsy commented Mar 6, 2025 •
edited by jrchamp
Loading

  • Updated zoom_calendar_item_update() to check if the occurrence has been "deleted"
  • If occurrence status is "deleted", do not add to calendar

Fixes #641

@cbounphengsy cbounphengsy added the bug Fixes problems or reduces technical debt label Mar 6, 2025
@cbounphengsy cbounphengsy requested a review from jrchamp March 6, 2025 15:45
@cbounphengsy cbounphengsy self-assigned this Mar 6, 2025
jrchamp
jrchamp reviewed Mar 6, 2025
View reviewed changes
Copy link
Collaborator

@jrchamp jrchamp left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Excellent find! I did not expect that Zoom would have a "status => deleted" for deleted occurrences.

@jrchamp jrchamp merged commit d70bcb8 into main Mar 6, 2025
14 checks passed
@jrchamp jrchamp deleted the fix/calendar_entries branch March 6, 2025 16:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@jrchamp jrchamp jrchamp approved these changes

Assignees

@cbounphengsy cbounphengsy

Labels
bug Fixes problems or reduces technical debt
Projects
mod_zoom Workflow
Status: No status
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Remove calendar entries
2 participants
@cbounphengsy @jrchamp