Project

General

Profile

Actions

Feature #12332

closed

Limit length of email subject triggered by event activity when activity is linked to multiple groups

Added by Matt Gold about 4 years ago. Updated about 4 years ago.

Status:
Resolved
Priority name:
Low
Assignee:
Category name:
Events
Target version:
Start date:
2020-01-27
Due date:
% Done:

0%

Estimated time:
Deployment actions:

Description

I received a notification of a new event creation; because it was posted to multiple groups, it had a very long subject line:

Stefano Morello created the event The Social Backend: Community-Driven Digital Archives and Exhibits in the groups GC Events and Workshops, DARC (Digital Archive Research Collective), Digital Humanities Initiative, GC Digital Research Institute - January 2019, GC Digital Research Institute January 2018, GC Digital Research Institute – January 2020, Provost’s Digital Innovation Grant Recipients.

(screenshot attached)

Can we limit the number of groups mentioned in the subject line of the email and/or find a way to shorten the title in situations like this?


Files


Related issues

Related to CUNY Academic Commons - Feature #12505: Email subject linesResolvedLuke Waltzer2020-03-04

Actions
Actions #2

Updated by Boone Gorges about 4 years ago

  • Tracker changed from Bug to Feature
  • Subject changed from Limit event titles to Limit length of email subject triggered by event activity when activity is linked to multiple groups
  • Status changed from Assigned to Reporter Feedback
  • Target version set to 1.17.0

How? Perhaps we can truncate the list after a certain number. If there's one or two groups, we list them. If there's three or more, list the first two, with 'and x more groups'? So: 'Stefano Morello created the event The Social Backend: Community-Driven Digital Archives and Exhibits in the groups GC Events and Workshops, DARC (Digital Archive Research Collective), and 3 other groups'. Or the threshold could be dynamic, depending on the number of characters in the group names: if each name is short, we might show 4 or 5 before truncation; if they're longer, we might show just one.

Actions #3

Updated by Raymond Hoh about 4 years ago

How about we keep it simple and if an event is posted in more than one group, we say "in multiple groups" instead?

The email notification / activity content can include the list of groups if it doesn't already. I can't remember if it does or not.

Actions #4

Updated by Boone Gorges about 4 years ago

Yes, that is a good idea, Ray. The tricky thing in that case, though, is that the text of the activity action printed in the email body will be different from what's used for the email notification. So we'd need to intercept the email at the level of BPGES rather than simply changing the text of the activity item generated by bp-event-organiser. This is a bit harder technically, but I think you're probably right that it's the best solution.

Actions #5

Updated by Boone Gorges about 4 years ago

Actions #6

Updated by Raymond Hoh about 4 years ago

  • Status changed from Reporter Feedback to Staged for Production Release
  • Assignee changed from Boone Gorges to Raymond Hoh
  • Target version changed from 1.17.0 to 1.16.8

This ticket intersected with #12505, so was able to fix this at the same time!

Code reference - https://github.com/cuny-academic-commons/cac/commit/b81f3af9b208f608b759c30ed5571e35070b9b9d

Actions #7

Updated by Boone Gorges about 4 years ago

  • Status changed from Staged for Production Release to Resolved
Actions

Also available in: Atom PDF