Skip to content

bugfix and minor gitignore update - #100

Open
zacs wants to merge 2 commits into
timeframe:mainfrom
zacs:claude/eager-hamilton-974t43
Open

zacs wants to merge 2 commits into
timeframe:mainfrom
zacs:claude/eager-hamilton-974t43

Conversation

@zacs

@zacs zacs commented Nov 2, 2026

Copy link
Copy Markdown
Contributor

Fixes #99, and also adds a couple paths to gitignore that are helpful when running in standalone container mode.

Running with a ./data bind mount (instead of the named volume in
docker-compose.yml) puts the Postgres data directory inside the repo.
It is owned by the container's postgres user, so the next
`docker compose up --build` fails with:

  failed to solve: error from sender: open .../data/pgdata: permission denied

Exclude data/ and pgdata/ from the build context, and keep
docker-compose.override.yml (the natural place for personal settings
such as the Home Assistant token) out of both the image and git.
Also ignore /data/ in git: it holds the SECRET_KEY_BASE file, and
committing or cleaning it away would make the encrypted data unreadable.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01D3eqQ5cKPh9Sm6H9E7YWQu
Home Assistant now includes a `status` field (confirmed/tentative) on
calendar events. fetch_calendars keeps every field HA returns apart from
a fixed list, and calendar_events splats the rest into DeviceEvent.new,
so the new field raises

  ArgumentError (unknown keyword: :status)

That takes down the Events page, the Status page (which lists calendar
events), and device rendering, which also goes through calendar_events.

Slice each event to the keywords DeviceEvent#initialize accepts before
constructing it. Doing this on read also covers events already cached
before the Home Assistant upgrade, and future HA fields can't break it.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01D3eqQ5cKPh9Sm6H9E7YWQu

This branch has not been deployed

No deployments
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.

Recent HA versions' calendars break timeframe

2 participants