Opened 6 months ago

Last modified 6 months ago

#3097 new defect

Discourse trac SAC email subjects are too long

Reported by: neteler Owned by: sac-tickets@…
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 robe, 6 months ago

Component: SysAdminSysAdmin/Discourse

comment:2 by robe, 6 months ago

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.

comment:3 by robe, 6 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.

in reply to:  2 comment:4 by neteler, 6 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 jsanz, 6 months ago

Yeah this is bad

https://i.imgur.com/WXdcK4V.png

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 or SAC/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 robe, 6 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 robe, 6 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 robe, 6 months ago

Cc: strk 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 robe, 6 months ago

okay I changed it to

%{optional_cat}%{topic_title}

See how you like that.

comment:10 by neteler, 6 months ago

Your test message looks much better, great! I'll watch out for more messages comin' in...

comment:11 by strk, 6 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 strk, 6 months ago

Type: taskdefect
Note: See TracTickets for help on using tickets.