-
Notifications
You must be signed in to change notification settings - Fork 4
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
Conversation System: Add email notifications for comments #272
Comments
I did my diligence research to discover the main jobs-to-be-done from a user perspective:
To limit our purpose but making the experience better, I think we should narrow the solution to the first item and automatically notify the comment authors only about replies to their comments. The other two items from the list involve a higher level of complexity and 3rd party plugins might be needed to control all the use-cases. I draw a comment notification scheme to see which users receive notifications and when. You can find it here and you can explore further variations if needed: Starting from this motivation, I propose a set of guidelines that the solution should aim for. Emails notifications triggers:
Emails content:
@vladolaru That’s my call on this matter. Now it’s up to you to decide if we should use a plugin or (better) you can load the functionality from an existing one. Let me know when you need my additional input or feedback. |
@georgeolaru Very good analysis and wrap-up. I'll get to it. |
@vladolaru Just remembered a mention here regarding the Falcon project for Email Notifications (by Human Made) — maybe it helps. |
As a rule of thumb, for communication to succeed, both the sender and the receiver must be able to maintain a flow of information from and to each other.
In terms of an online conversation based on asynchronous communication, there should be a way to notify each party that the other has responded to its message.
This is usually done through email notifications that encourage the receiver to continue the conversation.
Plugins available:
Issue related to #268
The text was updated successfully, but these errors were encountered: