fix: remove redundant update of a message status

monta-app · Jan 25, 2022 · 6590f0a · 6590f0a
1 parent 36c2bb2
commit 6590f0a
Show file tree

Hide file tree

Showing 2 changed files with 0 additions and 16 deletions.
diff --git a/courier/sms.go b/courier/sms.go
@@ -109,13 +109,5 @@ func (c *courier) dispatchSMS(ctx context.Context, msg Message) error {
 		return errors.New(http.StatusText(res.StatusCode))
 	}
 
-	if err := c.deps.CourierPersister().SetMessageStatus(ctx, msg.ID, MessageStatusSent); err != nil {
-		c.deps.Logger().
-			WithError(err).
-			WithField("message_id", msg.ID).
-			Error(`Unable to set the message status to "sent".`)
-		return err
-	}
-
 	return nil
 }
diff --git a/courier/smtp.go b/courier/smtp.go
@@ -169,14 +169,6 @@ func (c *courier) dispatchEmail(ctx context.Context, msg Message) error {
 		return errors.WithStack(err)
 	}
 
-	if err := c.deps.CourierPersister().SetMessageStatus(ctx, msg.ID, MessageStatusSent); err != nil {
-		c.deps.Logger().
-			WithError(err).
-			WithField("message_id", msg.ID).
-			Error(`Unable to set the message status to "sent".`)
-		return err
-	}
-
 	c.deps.Logger().
 		WithField("message_id", msg.ID).
 		WithField("message_type", msg.Type).