Skip to content

delete_event returns state, JMAPMethodError.properties, optional calendar_id - #76

Merged
SashankBhamidi merged 1 commit into
mainfrom
feat/delete-state-properties-multicalendar
Oct 24, 2026
Merged

SashankBhamidi merged 1 commit into
mainfrom
feat/delete-state-properties-multicalendar

Conversation

@SashankBhamidi

Copy link
Copy Markdown
Member

delete_event returns (old_state, new_state), not None. Both servers already send this on destroy.

JMAPMethodError gains properties, from the SetError's own list when the server sends one.

create_event/send_invite take calendar_id=None to keep the event's own calendarIds instead of collapsing to one. Stalwart allows an event in two calendars; Cyrus doesn't. Raises ValueError if the event has none.

@SashankBhamidi
SashankBhamidi removed the request for review from stevepiercy October 24, 2026 17:26
@codecov

codecov Bot commented Oct 24, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

@read-the-docs-community

Copy link
Copy Markdown

@SashankBhamidi
SashankBhamidi merged commit 4413375 into main Oct 24, 2026
14 checks passed
@SashankBhamidi
SashankBhamidi deleted the feat/delete-state-properties-multicalendar branch October 24, 2026 17:27
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