oiupoyt/discordo
1
1
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity

refactor(notifications): remove redundant DM check and use MessageMentions flags only

Browse source
This commit is contained in:
ayn2op 2025-12-29 20:26:33 -05:00
parent f42884ec07
commit 92eb4425a1
1 changed files with 1 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

3
internal/notifications/notifications.go
View file

@ -65,8 +65,7 @@ func Notify(state *ningen.State, message *gateway.MessageCreateEvent, cfg *confi
slog.Info("failed to get profile image from cache for notification", "err", err, "hash", hash)
}
isChannelDM := channel.Type == discord.DirectMessage || channel.Type == discord.GroupDM
shouldChime := cfg.Notifications.Sound.Enabled && (!cfg.Notifications.Sound.OnlyOnPing || (isChannelDM || mentions.Has(ningen.MessageMentions|ningen.MessageNotifies)))
shouldChime := cfg.Notifications.Sound.Enabled && (!cfg.Notifications.Sound.OnlyOnPing || mentions.Has(ningen.MessageMentions|ningen.MessageNotifies))
if err := sendDesktopNotification(title, content, imagePath, shouldChime, cfg.Notifications.Duration); err != nil {
return err
}