Skip to content

Ignore unknown calendar event fields - #98

Closed
pudgeball wants to merge 1 commit into
timeframe:mainfrom
pudgeball:fix/calendar-event-status
Closed

pudgeball wants to merge 1 commit into
timeframe:mainfrom
pudgeball:fix/calendar-event-status

Conversation

@pudgeball

Copy link
Copy Markdown

Home Assistant's calendar API now returns a status field on events (e.g. Google Calendar's confirmed/tentative). calendar_events splatted every field into DeviceEvent.new, so the unknown keyword raised ArgumentError and the Events page, Status page and device renders all returned 500.

Only pass through the keywords DeviceEvent#initialize accepts, and add a regression test with a status field.

Claude-Session: https://claude.ai/code/session_0157TCwfdp7oDPLpdzpdDM6q

Home Assistant's calendar API now returns a `status` field on events
(e.g. Google Calendar's confirmed/tentative). calendar_events splatted
every field into DeviceEvent.new, so the unknown keyword raised
ArgumentError and the Events page, Status page and device renders all
returned 500.

Only pass through the keywords DeviceEvent#initialize accepts, and add
a regression test with a `status` field.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_0157TCwfdp7oDPLpdzpdDM6q
@pudgeball pudgeball closed this Nov 1, 2026
@pudgeball
pudgeball deleted the fix/calendar-event-status branch November 1, 2026 22:20
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