Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Enhancement]: Use message Subject as subject of DNN Messaging Notification Emails #6176

Open
2 of 4 tasks
Timo-Breumelhof opened this issue Oct 25, 2024 · 7 comments · May be fixed by #6199
Open
2 of 4 tasks

[Enhancement]: Use message Subject as subject of DNN Messaging Notification Emails #6176

Timo-Breumelhof opened this issue Oct 25, 2024 · 7 comments · May be fixed by #6199

Comments

@Timo-Breumelhof
Copy link
Contributor

Is there an existing issue for this?

  • I have searched the existing issues

Description of problem

When a module uses DNN messaging, the email being sent out always have a generic subject.
See:

image

Although DNN messaging is a great way to notify users and also send out email notifications, the fact that the subject is always the same is not ideal.
As an example the forum on DNN Community.
I get a lot of emails sent out by the community site.
During a busy period I prefer to only open the messages that are Forum replies to a thread I contributed to.
Currently you have to open an email to see what it's about, which is not very efficient.

Description of solution

IMO if a message has a subject, we should also use that as the Subject for the email notification.

Description of alternatives considered

I have been thinking of reasons why you would not want this behavior but I cant think of any TBH..

Anything else?

We would really like this change for the Community Forums Module :-)

Do you be plan to contribute code for this enhancement?

  • Yes

Would you be interested in sponsoring this enhancement?

  • Yes

Code of Conduct

  • I agree to follow this project's Code of Conduct
@Timo-Breumelhof Timo-Breumelhof changed the title [Enhancement]: Use message Subject as subject of Notification emails [Enhancement]: Use message Subject as subject of DNN Messaging Notification emails Oct 25, 2024
@Timo-Breumelhof
Copy link
Contributor Author

@johnhenley Issue :-)

@Timo-Breumelhof Timo-Breumelhof changed the title [Enhancement]: Use message Subject as subject of DNN Messaging Notification emails [Enhancement]: Use message Subject as subject of DNN Messaging Notification Emails Oct 25, 2024
@MarietteNL
Copy link

A daily or weekly digest of messages would be nice.

@Timo-Breumelhof
Copy link
Contributor Author

@MarietteNL do you mean for DNN or the Forums?

@johnhenley
Copy link
Contributor

@johnhenley Issue :-)

@Timo-Breumelhof I can do the code changes and put in a pull request. However, I'd like to get agreement beforehand that this enhancement is acceptable to the community. We would like it so we can enhance forums notifications at some point to use DNN notifications rather than direct emails.

@johnhenley
Copy link
Contributor

A daily or weekly digest of messages would be nice.

@MarietteNL if you're speaking about forums, we've had a long-standing request to revive that feature which was part of the "enterprise" forums I think. DNNCommunity/Dnn.CommunityForums#163

If you're speaking about a general digest of DNN notifications/messages, that is already available in the platform if I remember correctly. And that is likely what we want to leverage for forums to provide a digest rather than reinventing our own.

@Timo-Breumelhof
Copy link
Contributor Author

@johnhenley speaking of digest.. You can only replace the title when there's only one message included..

@johnhenley
Copy link
Contributor

@johnhenley speaking of digest.. You can only replace the title when there's only one message included..

Yes I realize that.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants