gdpelican
March 29, 2016, 1:20am
36
Spec is here:
I’m making this proposal in response to the following topics:
I think the vast majority of the problems & shortcomings discussed in the topics above would be solved by slightly more granular options for the “mailing list mode”. Now that we have the option to [disable mailing list mode] altogether, I think it’s okay to give it a space of its own.
I wanna go from this:
(yes that’s a mockup but it accurately represents our current layout)
[image]
(also note how “Send me an email for eve…
PR is here:
discourse:master ← gdpelican:feature/granular-mailing-list-mode
opened 09:02AM - 09 Mar 16 UTC
As discussed here:
https://meta.discourse.org/t/more-granular-mailing-list-mode/… 38107
As it looks now:
TODO:
- Determining when / how often to send grouped emails
- Designing email summary template
Would particularly like some feedback on the formula for email guesstimation here:
https://github.com/discourse/discourse/pull/4068/files#diff-08765a4c969575258af3892abcfa35a7R69
Currently it's looking at the number of posts created over the past week / month (with the past week average weighted 3x the past month average). Could likely do better around removing private messages, whispers, other stuff not visible to the user, etc., but figured I'd start with what was laying around.
Related line of code:
def mailing_list_mode_frequency_estimate
# average of last 30 days of posts
@mailing_list_mode_frequency_estimate ||= About.fetch_stats[:posts_30_days].to_f / 30
end
As stated it’s really cheap to calc and a mild overestimate, but maybe that’s alright because it’s better to surprise folks with fewer emails rather than surprise them with more.
7 Likes