Opened 14 years ago
Closed 8 years ago
#8738 closed enhancement (wontfix)
Ticketlink in text mail is not recognized
Reported by: | anonymous | Owned by: | |
---|---|---|---|
Priority: | normal | Component: | AnnouncerPlugin |
Severity: | normal | Keywords: | configuration link |
Cc: | Ryan J Ollos, Robert Corsaro | Trac Release: | 0.12 |
Description
Generated text mails contains the word URL: followed by the link. Mail programs (e.g. Outlook) do not recognize it as a link. The same applies for the project link. The corresponding template is in this file: http://trac-hacks.org/browser/announcerplugin/trunk/announcer/templates/ticket_email_plaintext.txt
Attachments (1)
Change History (5)
Changed 13 years ago by
Attachment: | announcerplugin-no-URL-in-plaintext-links.patch added |
---|
comment:1 Changed 13 years ago by
Priority: | normal → high |
---|
Noticed the same problem, using the iPad email client.
The attached patch fixes it for me.
As the format without URL: is what trac notifications themselves use, my opinion is that this should be applied ASAP.
If there is a good reason to have the URL: prefix in some setups, I suggest making it a configuration option, and prepending URL: to the relevant links in the formatters code, instead of having it in the template.
comment:2 Changed 12 years ago by
Cc: | Ryan J Ollos Robert Corsaro added; anonymous removed |
---|---|
Keywords: | configuration link added |
Owner: | changed from Robert Corsaro to Steffen Hoffmann |
Priority: | high → normal |
This is bad in terms of usability, right. But simply put we have too many other, more critical issues to deal with for now, sorry. I'm forced to skip this and come back later.
For the patch itself, it looks like no-one seems to see issues with removing URL:
-prefix unconditionally. So I guess, that we'll do that in the end. Thanks for contributing this suggestion.
comment:3 Changed 8 years ago by
Owner: | Steffen Hoffmann deleted |
---|
comment:4 Changed 8 years ago by
Resolution: | → wontfix |
---|---|
Status: | new → closed |
Please upgrade to Trac 1.2, which has integrated the core of AnnouncerPlugin. Please raise the issue on the trac:MailingList if you encounter the issue with Trac 1.2.
remove URL: from plaintext templates