Skip to content

Fix Proton Calendar CalDAV sync#6

Open
rupert648 wants to merge 2 commits into
acheong08:mainfrom
rupert648:main
Open

Fix Proton Calendar CalDAV sync#6
rupert648 wants to merge 2 commits into
acheong08:mainfrom
rupert648:main

Conversation

@rupert648

@rupert648 rupert648 commented Feb 6, 2026

Copy link
Copy Markdown

Hi! Massively appreciate this tool. This fixes a few bugs/adds a few improvements I found useful when using the calendar sync, happy to open issues and break this down further.

  • Handle empty Author field in events (imported/birthday events)
  • Filter out birthday/holiday calendars by name (Proton returns Type=0 for all)
  • Fix CalendarEventCardType.Encrypted() to correctly identify Type=1 as encrypted
  • Skip malformed events instead of failing entire sync
  • Add date range filter to fetch all events (5 years past/future)
  • Add pagination to ListCalendarEvents to fetch beyond 100 event limit
  • Fix event creation: use errors.As() for wrapped APIError detection

- Handle empty Author field in events (imported/birthday events)
- Filter out birthday/holiday calendars by name (Proton returns Type=0 for all)
- Fix CalendarEventCardType.Encrypted() to correctly identify Type=1 as encrypted
- Skip malformed events instead of failing entire sync
- Add date range filter to fetch all events (5 years past/future)
- Add pagination to ListCalendarEvents to fetch beyond 100 event limit
- Fix event creation: use errors.As() for wrapped APIError detection
@hacknus

hacknus commented Feb 16, 2026

Copy link
Copy Markdown

I opened a PR (rupert648#1) on this PR to address #7 partially (contacts/cardDAV are still not implemented)

* update for apple

* fix

* implement push

* fix

* fix

* fix
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.

2 participants