Change email notifier subject line #11
Conversation
Grouped/Threaded conversations based on subject line make the current email alerts hard to follow - at least they do in GMail!
|
Good, but we could add description instead of ID, what do you think? |
|
Our descriptions are generally short (they're from Icinga) so I have no problem with that. The description field limit is 100 chars though, should we truncate that in the email? |
|
Actually its not that long - I'll just change it. |
|
Ha, so it turns out our descriptions are being silently truncated (long hostnames again), so the 100 char limit may not be enough! Ok, how about this: we increase the description character limit on the model, and truncate it to 100 characters in the email subject? |
|
This works for me perfectly. If you have any ideas, we could talk about it on #openduty on Freenode. |
|
Ok, changes pushed - made the truncation optional and tuneable. I'll keep IRC in mind, thanks! |
Grouped/Threaded conversations based on subject line make the current email alerts hard to follow - at least they do in GMail!
I hope you don't mind many small pull requests - I'm doing it this way so you can cherry pick the small changes I'm making to scratch some itches as I find them.