Feature #6628
closed
Improvements in truncate email
100%
Description
Rather than hunting for a correct regexp, why can't we simply set a line -
-- Reply above this line --
The above line is at the very top of the message, and the email parsing script will simply chop off the body after this line. Isn't this simpler than the existing feature. I believe this is how most software that do 'reply by email' work.
To make it backward compatible, this could be an additional option
Files
Related issues
Updated by Holger Just about 14 years ago
There is already the possibility to define delimiter strings. See Administration > Settings -> Incoming emails. To include the line you chose, you might have to update the template for sent mails.
Updated by Ashwin Date about 14 years ago
Thanks a lot for the idea Holger. I think that should work perfectly. Will try tomorrow and post results.
Thats what happens when you try to think when it's sleep time...
Updated by Ashwin Date about 14 years ago
- File redmine-email.jpg redmine-email.jpg added
- File update-with-garbage.jpg update-with-garbage.jpg added
Ok I tried this. Here's what I did
In the settings, changed Truncate email after.... to
\-\-Reply above this line\-\-
Edited the config/locales/en.yml and changed
text_issue_updated: "Issue {{id}} has been updated by {{author}}."
to
text_issue_updated: "--Reply above this line--<br />Issue {{id}} has been updated by {{author}}."
The email that I received had that line as expected, but when it got added as a comment, it had all the text underneath. Have attached screenshots of how the email looks and how the issue update looks.
Note that I also tried the following in the en.yml
text_issue_updated: "--Reply above this line--\nIssue {{id}} has been updated by {{author}}."
text_issue_updated: "--Reply above this line-- Issue {{id}} has been updated by {{author}}."
I've set the notification emails to plain text
Updated by Ashwin Date about 14 years ago
Note that the
\-\-Reply above this line\-\-
above should have been
--Reply above this line--
I don't have the slashes either in the settings or the en.yml
Updated by Eric Davis about 14 years ago
- Status changed from New to Closed
- Assignee set to Eric Davis
- Target version set to 1.1.0
- % Done changed from 0 to 100
- Resolution set to Fixed
I've updated the email truncation in r4361 and r4362 to support reply quotes in front of the delimiter string (> ) and to allow setting a standard header for all emails.
Ashwin: with this you can now set your "Truncate emails" to "--Reply above this line--" and add that to your Email Header (Admin > Settings > Email Notifications and Incoming Emails).