Don't add default reminder to Google if email notification(s) set.

Fixes #1156
phw198 · Jan 23, 2021 · ef6aa86 · ef6aa86
1 parent 37f7274
commit ef6aa86
Showing 1 changed file with 3 additions and 2 deletions.
diff --git a/src/OutlookGoogleCalendarSync/GoogleOgcs/GoogleCalendar.cs b/src/OutlookGoogleCalendarSync/GoogleOgcs/GoogleCalendar.cs
@@ -785,7 +785,7 @@ public Event UpdateCalendarEntry(AppointmentItem ai, Event ev, ref int itemModif
                         } //if Google reminder found
                     } //foreach reminder
 
-                } else { //no Google reminders set
+                } else { //no Google popup reminders set
                     if (ai.ReminderSet && OKtoSyncReminder) {
                         sb.AppendLine("Reminder: nothing => " + ai.ReminderMinutesBeforeStart);
                         ev.Reminders.UseDefault = false;
@@ -795,7 +795,8 @@ public Event UpdateCalendarEntry(AppointmentItem ai, Event ev, ref int itemModif
                         ev.Reminders.Overrides = new List<EventReminder>();
                         ev.Reminders.Overrides.Add(newReminder);
                         itemModified++;
-                    } else {
+
+                    } else if (ev.Reminders.Overrides == null) { //No Google email reminders either
                         Boolean newVal = OKtoSyncReminder ? Settings.Instance.UseGoogleDefaultReminder : false;
 
                         //Google bug?! For all-day events, default notifications are added as overrides and UseDefault=false