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.