Project

General

Profile

Actions

Feature #6628

closed

Improvements in truncate email

Added by Ashwin Date over 13 years ago. Updated over 13 years ago.

Status:
Closed
Priority:
Normal
Assignee:
Category:
Email receiving
Target version:
Start date:
2010-10-11
Due date:
% Done:

100%

Estimated time:
Resolution:
Fixed

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

redmine-email.jpg (37.5 KB) redmine-email.jpg This is the email that I get Ashwin Date, 2010-10-20 16:44
update-with-garbage.jpg (50.6 KB) update-with-garbage.jpg This is how the update looks on the issue page Ashwin Date, 2010-10-20 16:44

Related issues

Related to Redmine - Feature #2852: Clean Incoming Email of quoted text "----- Reply above this line ------"ClosedEric Davis2009-02-27

Actions
Actions #1

Updated by Holger Just over 13 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.

Actions #2

Updated by Ashwin Date over 13 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...

Actions #3

Updated by Ashwin Date over 13 years ago

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

Actions #4

Updated by Ashwin Date over 13 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

Actions #5

Updated by Eric Davis over 13 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).

Actions

Also available in: Atom PDF