Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Sync fails if gmail event notifications contain only "Email events" #1358

Closed
paultgordon opened this issue Dec 16, 2021 · 1 comment
Closed
Labels
bug Something's not working right duplicate Already covered by another issue

Comments

@paultgordon
Copy link

Checklist:-

  • Upgrade to the latest release (alpha if available) latest release
  • Reproduce the issue
  • Complete the information below
  • Provide your OGcalsync.log file (see how-to)

OGCS Version: v2.9.2.0
Edition: Fails on both
Sync Direction: O->G

Bug Description
Doing a sync O->G when your gmail calendar has only email events (not popup events) causing the sync to fail.

Steps to Reproduce the Issue

  1. Change gmail calendar settings to have email only events like photo attached. Instructions on how to change reminder settings are described well at: https://webapps.stackexchange.com/a/112905

Any other information
I determined the issue by stepping through the code. This is caused by the issue that in:

GoogleCalendar.cs::getCalendarSettings() -> cal.DefaultReminders.Where(x => x.Method.Equals("popup")).OrderBy(x => x.Minutes.Value).First().Minutes.Value_

The .Where does not return any items so the .First throws an exception.

Any additional information that may help troubleshoot the issue.
📎 Attach your OGcalsync.log file.
OGcalsync.log
GoogleEventNotifSettings

@paultgordon paultgordon added the bug Something's not working right label Dec 16, 2021
@phw198
Copy link
Owner

phw198 commented Jan 8, 2022

Thanks - though this is a duplicate of #1344 which has a hotfix posted.

@phw198 phw198 closed this as completed Jan 8, 2022
@phw198 phw198 added the duplicate Already covered by another issue label Jan 8, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something's not working right duplicate Already covered by another issue
Projects
None yet
Development

No branches or pull requests

2 participants