FEATURE: Use group SMTP settings for sending user notification emails (initial) (PR #13220)

GitHub

The title of this pull request changed from “Feature/use group smtp settings for sending user notification emails” to "FEATURE: Use group SMTP settings for sending user notification emails (initial)

mixed on merging it in with this warning, I guess if we really want this then:

if ENV[‘DISCOURSE_VERBOSE_GROUP_MESSAGES’] … end

Get rid of that block later on

looks pretty safe to merge to me.

I don’t intend to leave this in there once it goes to Meta/other places, this is mostly for my benefit when I do some testing on our hosted site. I really doubt anyone has this setting enabled right now for it to show in our logs

OK, this is very minor, go ahead and merge as is.