소스 검색

fix: do not save determination id until after successfully updating calendar

Andreas Stokholm 1 년 전
부모
커밋
cb61ba1740
1개의 변경된 파일3개의 추가작업 그리고 1개의 파일을 삭제
  1. 3 1
      FreeAPS/Sources/Services/Calendar/CalendarManager.swift

+ 3 - 1
FreeAPS/Sources/Services/Calendar/CalendarManager.swift

@@ -216,7 +216,6 @@ final class BaseCalendarManager: CalendarManager, Injectable {
         if determinationId == previousDeterminationId {
             return
         }
-        previousDeterminationId = determinationId
 
         let glucoseIds = await fetchGlucose()
 
@@ -298,6 +297,9 @@ final class BaseCalendarManager: CalendarManager, Injectable {
 
             try eventStore.save(event, span: .thisEvent)
 
+            // Save the determination ID so we can avoid updating calendar if this determination comes back
+            previousDeterminationId = determinationId
+
         } catch {
             debugPrint(
                 "\(DebuggingIdentifiers.failed) \(#file) \(#function) Failed to create calendar event: \(error.localizedDescription)"