From e862a281d990fce370a56f44fe0da9be435da90c Mon Sep 17 00:00:00 2001 From: Mo Tarbin Date: Sun, 7 Jul 2024 19:53:08 -0400 Subject: [PATCH] Add time field to FrequencyMetadata struct --- internal/chore/model/model.go | 1 + internal/chore/scheduler.go | 12 ++++++++++++ 2 files changed, 13 insertions(+) diff --git a/internal/chore/model/model.go b/internal/chore/model/model.go index 90f6c29..3fb7a92 100644 --- a/internal/chore/model/model.go +++ b/internal/chore/model/model.go @@ -47,6 +47,7 @@ type FrequencyMetadata struct { Days []*string `json:"days,omitempty"` Months []*string `json:"months,omitempty"` Unit *string `json:"unit,omitempty"` + Time string `json:"time,omitempty"` } type NotificationMetadata struct { diff --git a/internal/chore/scheduler.go b/internal/chore/scheduler.go index c1ff48d..5413447 100644 --- a/internal/chore/scheduler.go +++ b/internal/chore/scheduler.go @@ -22,6 +22,7 @@ func scheduleNextDueDate(chore *chModel.Chore, completedDate time.Time) (*time.T if chore.FrequencyType == "once" { return nil, nil } + if chore.NextDueDate != nil { // no due date set, use the current date @@ -29,6 +30,17 @@ func scheduleNextDueDate(chore *chModel.Chore, completedDate time.Time) (*time.T } else { baseDate = completedDate.UTC() } + if chore.FrequencyType == "day_of_the_month" || chore.FrequencyType == "days_of_the_week" || chore.FrequencyType == "interval" { + // time in frequency metadata stored as RFC3339 format like `2024-07-07T13:27:00-04:00` + // parse it to time.Time: + t, err := time.Parse(time.RFC3339, frequencyMetadata.Time) + if err != nil { + return nil, fmt.Errorf("error parsing time in frequency metadata") + } + // set the time to the time in the frequency metadata: + baseDate = time.Date(baseDate.Year(), baseDate.Month(), baseDate.Day(), t.Hour(), t.Minute(), 0, 0, t.Location()) + + } if chore.IsRolling && chore.NextDueDate.Before(completedDate) { // we need to check if chore due date is before the completed date to handle this senario: // if user trying to complete chore due in future (multiple time for insance) due date will be calculated