Skip to content

fix(calendar): preserve RRULE and set timezone for recurring updates#392

Merged
steipete merged 3 commits intoopenclaw:mainfrom
salmonumbrella:fix/issue-347-recurring-timezone
Mar 7, 2026
Merged

fix(calendar): preserve RRULE and set timezone for recurring updates#392
steipete merged 3 commits intoopenclaw:mainfrom
salmonumbrella:fix/issue-347-recurring-timezone

Conversation

@salmonumbrella
Copy link
Copy Markdown
Contributor

@salmonumbrella salmonumbrella commented Mar 3, 2026

Summary

  • prevent --rrule values from being comma-split by setting sep:"none"
  • ensure recurring update patches include start/end.timeZone (from provided values or existing event data)
  • add regression tests for RRULE payload integrity and recurring update timezone enrichment

Fixes #347

Testing

  • go test ./internal/cmd -run 'TestCalendarUpdateCmd_RecurrenceFillsMissingTimezone|TestCalendarCreateCmd_RecurringOffsetTimezoneFallback'
  • go test ./internal/cmd
  • make test

@steipete steipete force-pushed the fix/issue-347-recurring-timezone branch from 4f138a7 to f6de744 Compare March 7, 2026 22:47
@steipete steipete merged commit 248623f into openclaw:main Mar 7, 2026
1 check passed
@steipete
Copy link
Copy Markdown
Collaborator

steipete commented Mar 7, 2026

Landed via rebase onto main.

  • Gate: go test ./internal/cmd -run 'TestCalendarUpdateCmd_RecurrenceFillsMissingTimezone|TestCalendarCreateCmd_RecurringOffsetTimezoneFallback' && make test
  • Land commit: f6de744
  • Merge commit: 248623f

Thanks @salmonumbrella!

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.

Bug: Missing time zone definition for start time when creating recurring events

2 participants