Opened 12 months ago
Last modified 12 months ago
#3097 new defect
Discourse trac SAC email subjects are too long
Reported by: | neteler | Owned by: | |
---|---|---|---|
Priority: | normal | Milestone: | Unplanned |
Component: | SysAdmin/Discourse | Keywords: | discourse, email |
Cc: | strk |
Description
For testing, I subscribed to the discourse SAC category and got a tons of emails looking like this:
- [OSGeo] [System Administration Committee/SAC tickets] [SAC] [OSGeo] #893: [new icons] Icons for vector processing tools in QGIS
- [OSGeo] [System Administration Committee/SAC tickets] [SAC] [OSGeo] #1369: Register osgeo.org.ua
- [OSGeo] [System Administration Committee/SAC tickets] [OSGeo] #3094: Unable to add members to a project
- ...
There's a bit of duplication in the subject - and to spell out SAC is probably not needed :)
If anyone knows how to set the the discourse generated subjects properly I'd be grateful.
Thanks Markus
Change History (12)
comment:1 by , 12 months ago
Component: | SysAdmin → SysAdmin/Discourse |
---|
follow-up: 4 comment:2 by , 12 months ago
comment:3 by , 12 months ago
I reduced the main sac category to SAC, so that should reduce the subject line a bit. I haven't investigated to see how to get rid of parent category from the subject line.
comment:4 by , 12 months ago
Replying to robe:
I promoted you to discourse admin @neteler so you should be able to see more. Not sure where the setting is myself if there is one.
Thanks much, I'll try to find the settings.
comment:5 by , 12 months ago
Yeah this is bad
Some thoughts from left to right :_)
- Would it be acceptable to try to change the global setting in Discourse to shorten the first part of the mail so it is just
[OSGeo]
instead of[OSGeo Discourse]
. - New category name is better, but why not renaming the Discourse category to just "Trac" so it becomes
SAC/Trac
orSAC/Tickets
- The last
[OSGeo]
bit comes from Trac so I understand we don't want to change that since it would affect all components, right?
comment:6 by , 12 months ago
I'm fine with both. The OSGeo Discourse I guess it's because I changed the title, cause I got annoyed it saying OSGeo in some of the other things (not the email).
I recall seeing a setting somewhere in admin panel for that, so maybe we can just not use the title and hardcode the value there.
comment:7 by , 12 months ago
I found the setting - it's in https://discourse.osgeo.org/admin/site_settings/category/email
email subject:
which is currently set to:
[%{site_name}] %{optional_pm}%{optional_cat}%{topic_title}
So I guess we can get rid of optional_pm (which I assume is for primary category)
and also change [%{site_name}] to simply OSGeo. Or you think we even need that?
Maybe the category is sufficient.
comment:8 by , 12 months ago
Cc: | added |
---|
I'm also thinking maybe we don't need category as that would be part of the list-header thingy attribute. @strk you have an opinion on this.
It's too bad we don't have this option per category that it's a global setting. If there is a per category setting, I haven't found it.
comment:9 by , 12 months ago
okay I changed it to
%{optional_cat}%{topic_title}
See how you like that.
comment:10 by , 12 months ago
Your test message looks much better, great! I'll watch out for more messages comin' in...
comment:11 by , 12 months ago
I'm ok with removing the category tag as email headers contain enough info for any filter:
X-Discourse-Post-Id: 22015 X-Discourse-Topic-Id: 6547 X-Discourse-Category: SAC/SAC tickets List-ID: OSGeo Discourse | SAC SAC tickets <sac-tickets.sac.discourse.osgeo.org>
comment:12 by , 12 months ago
Type: | task → defect |
---|
I promoted you to discourse admin @neteler so you should be able to see more. Not sure where the setting is myself if there is one.