Clarification on enqueue failure

dannykopping · dannykopping · commit 0abd613bf2fc · 2024-08-05T13:55:35.000+02:00
Test fix

Signed-off-by: Danny Kopping &lt;danny@coder.com&gt;
diff --git a/coderd/notifications/enqueuer.go b/coderd/notifications/enqueuer.go
@@ -83,6 +83,11 @@ func (s *StoreEnqueuer) Enqueue(ctx context.Context, userID, templateID uuid.UUI
 		CreatedBy:              createdBy,
 	})
 	if err != nil {
+		// We have a trigger on the notification_messages table named `inhibit_enqueue_if_disabled` which prevents messages
+		// from being enqueued if the user has disabled them via notification_preferences. The trigger will fail the insertion
+		// with the message "cannot enqueue message: user has disabled this notification".
+		//
+		// This is more efficient than fetching the user's preferences for each enqueue, and centralizes the business logic.
 		if strings.Contains(err.Error(), ErrCannotEnqueueDisabledNotification.Error()) {
 			return nil, ErrCannotEnqueueDisabledNotification
 		}
