review comments

models/msgs.go

@@ -551,6 +551,9 @@ func InsertChildBroadcast(ctx context.Context, db Queryer, parent *Broadcast) (*
 	child.b.Text.Map = make(map[string]sql.NullString)
 	for lang, t := range child.b.Translations {
 		child.b.Text.Map[string(lang)] = sql.NullString{String: t.Text, Valid: true}
+		if len(t.Attachments) > 0 || len(t.QuickReplies) > 0 {
+			return nil, errors.Errorf("cannot clone broadcast with quick replies or attachments")
+		}
 	}
 
 	// insert our broadcast

tasks/schedules/cron.go

@@ -28,6 +28,9 @@ func StartCheckSchedules(mr *mailroom.Mailroom) error {
 		func(lockName string, lockValue string) error {
 			ctx, cancel := context.WithTimeout(context.Background(), time.Minute*5)
 			defer cancel()
+			// we sleep 1 second since we fire right on the minute and want to make sure to fire
+			// things that are schedules right at the minute as well (and DB time may be slightly drifted)
+			time.Sleep(time.Second * 1)
 			return checkSchedules(ctx, mr.DB, mr.RP, lockName, lockValue)
 		},
 	)