DNN.Blog icon indicating copy to clipboard operation
DNN.Blog copied to clipboard

Send comment notifications

Open Timo-Breumelhof opened this issue 6 years ago • 3 comments

Is your feature request related to a problem?

Social interaction of blog posts could be improved.

Describe the solution you'd like

I would be good for interaction if the blog post "owner" would get an email if there are comments on his / her post. It would also be nice if anybody commenting would (with opt-out) get an email when: A. Someone replies to their comment B. There are other comments

Somewhat related to #155

Timo-Breumelhof avatar Oct 30 '19 16:10 Timo-Breumelhof

I just looked into the code a bit... There is notification logic in the module, but it's wrapped in code that will seemingly never execute. When a comment is added, it's saved to the database, and that's followed by a check to see if the comment is approved or not. If not, then no notifications are sent.

As a follow-up test, I just got latest and setup a quick dev environment. It seems that my original suspiscion was correct. It'll never generate a notification. I think the IF is just reversed.

image

WillStrohl avatar Jan 12 '24 21:01 WillStrohl

I switched the IF statements, and the code runs without any issues, but the notification never is actually triggered even though the comments are set to auto-approve and I see they're approved in the database.

image

WillStrohl avatar Jan 12 '24 21:01 WillStrohl

Oh, and one more thing... I had SMTP enabled, and at no point did I see any emails get generated to any party.

WillStrohl avatar Jan 12 '24 22:01 WillStrohl