Commit694df11

committed

Clarification on enqueue failure

Test fixSigned-off-by: Danny Kopping <danny@coder.com>

1 parentd587308 commit694df11Copy full SHA for 694df11

File tree

-0

lines changed

-0

lines changed

Lines changed: 5 additions & 0 deletions

Original file line number	Diff line number	Diff line change
`@@ -83,6 +83,11 @@ func (s *StoreEnqueuer) Enqueue(ctx context.Context, userID, templateID uuid.UUI`
`83`	`83`	`CreatedBy:createdBy,`
`84`	`84`	`})`
`85`	`85`	`iferr!=nil {`
	`86`	+// We have a trigger on the notification_messages table named `inhibit_enqueue_if_disabled` which prevents messages
	`87`	`+// from being enqueued if the user has disabled them via notification_preferences. The trigger will fail the insertion`
	`88`	`+// with the message "cannot enqueue message: user has disabled this notification".`
	`89`	`+//`
	`90`	`+// This is more efficient than fetching the user's preferences for each enqueue, and centralizes the business logic.`
`86`	`91`	`ifstrings.Contains(err.Error(),ErrCannotEnqueueDisabledNotification.Error()) {`
`87`	`92`	`returnnil,ErrCannotEnqueueDisabledNotification`
`88`	`93`	`}`

Comments

(0)