I think duplication here isn’t too bad. Though keep in mind there are no post numbers so if you didn’t include it, you wouldn’t notice it wasn’t there. Not sure if it would be confusing or not. My bet is most everyone wouldn’t notice or care since it was above the previous replies.
Y’all are on the right track here; this is great to see. We get a ton of complaints about the lack of proper threading in the “previous replies”. It’s got to be obvious to the reader what the current notification is replying to, since inline replies aren’t possible.
90% of the time it won’t make much sense, but there are several cases, one being if the notification is being sent on an old thread - it could be anything from days to weeks old. To display the context as chosen by the user is important here, I feel. If they don’t want the context because they’ve got it in other emails, they can easily set it to ‘never’.
Re quoted, good point. If quoted, assume the relevant context has been included.
You are adding complexity here, I don’t think this is needed. Historical stuff is always fudgy in mailing list mode. However adding anything besides the actual body of the post in mailing list mode is like mixing water and oil, no mailing list does this, its orthogonal to mailing list designs.
But this is exactly the scenario that my users are asking for, to have context included when they use mailing list mode. They want to see what post something was in reply to. FWIW, I don’t agree with it, but if that’s what they want, then I think we should certainly consider it. What do other forums’ users think?
I think the options outlined above in the mailing list mode scenario fit this quite well.
At a bare minimum they want to see the immediate parent message to the notification. This is perfectly in line with most mailing list usage, because people can do inline quotes with “regular” mailing lists. Since that’s not possible with Discourse email use cases (for now) this is the only way I can see to provide that context.
I do have a concern here we are changing behavior of “context auto” . In the past “In Reply To” was not displayed so in mailing list mode you would not get that at all, post the change described every reply you get we will be inserting the “In Reply To” which forces people using mailing list to go to profile and change that setting.
I know people like @avdi who use mailing list mode pretty exclusively would get a bit of a surprise here. I guess the fix is a 2 second change from the user side.
Anyway we can discuss this ad-nauseum better just build something. Start with the drop down box, and make it work exactly the same for mailing list mode and non mailing list mode.
Any updates on the thinking here? We are about ready to migrate our 1500-member developer mailing list to Discourse. We already know the notifications are confusing to them. But, if notifications are going to change dramatically soon, I will try to hold off to introduce less pain.
Hi @downey - not much of an update from me, I’ve been swamped with stuff unfortunately. I’ve made a start on it, but not enough to start committing yet really. I do have a bit of free time this weekend which may help, but it’s still probably going to be a while till it’s committed, reviewed, and accepted.