Feature #5622

Allow version to be set via incoming email

Added by Wael Nasreddine over 7 years ago. Updated about 7 years ago.

Status:ClosedStart date:2010-05-30
Priority:NormalDue date:
Assignee:-% Done:

0%

Category:Email receiving
Target version:1.1.0
Resolution:Fixed

Description

Hello,

My boss always assign the target version when he opens issues by email, and I always have to edit all issues he (or I) opened via email to set the correct target version and remove it from the description manually.

It would be easier if redmine parses the "Version:" tag and assign it correctly.

Thanks

0001-Add-a-Version-keyword-to-receiving-email.-5622.patch Magnifier - Version keyword (4.69 KB) Wael Nasreddine, 2010-06-05 12:46

0001-Add-a-Version-keyword-to-receiving-email.-5622.patch Magnifier (4.69 KB) Wael Nasreddine, 2010-06-05 12:50

0001-Add-a-Version-keyword-to-receiving-email.-5622.patch Magnifier - Correctly indented files, please remove the rest (4.7 KB) Wael Nasreddine, 2010-06-05 12:55


Related issues

Related to Redmine - Feature #5594: Improve MailHandler's keyword handling Closed 2010-05-25

Associated revisions

Revision 4394
Added by Jean-Philippe Lang about 7 years ago

Makes MailHandler accept all issue attributes and custom fields that can be set/updated (#4071, #4807, #5622, #6110).

History

#1 Updated by Alexsandro Farias over 7 years ago

  • Status changed from New to Resolved

Wael Nasreddine wrote:

Hello,

My boss always assign the target version when he opens issues by email, and I always have to edit all issues he (or I) opened via email to set the correct target version and remove it from the description manually.

It would be easier if redmine parses the "Version:" tag and assign it correctly.

Thanks

#2 Updated by Felix Schäfer over 7 years ago

  • Status changed from Resolved to New

#3 Updated by Wael Nasreddine over 7 years ago

Ummmm Guys!! What's going on?

#4 Updated by Felix Schäfer over 7 years ago

Wael Nasreddine wrote:

Ummmm Guys!! What's going on?

It seems Alexsandro is trying out the redmine here rather than on the demo site and I reverted it, sorry for not stating it explicitly.

#5 Updated by Wael Nasreddine over 7 years ago

Felix Schäfer wrote:

Wael Nasreddine wrote:

Ummmm Guys!! What's going on?

It seems Alexsandro is trying out the redmine here rather than on the demo site and I reverted it, sorry for not stating it explicitly.

I kinda figured it out when I saw his Test defect :D, anyway no problem thanks for fixing it fast, hopefully it'll make it to redmine fast too :)

#6 Updated by Wael Nasreddine over 7 years ago

I've created a patch myself, the "Version" keyword is now parsed and correctly assigned, however the version must already exist, if the version does not already exist, it's ignored...

I plan on to extending it to create a Target Version if one doesn't exist, but I prefer if Eric or someone else do this part because I'm still too new to RoR...

#7 Updated by Wael Nasreddine over 7 years ago

I would appreciate it if someone would remove the first posted file because the unit test file is not indented correctly (using tabs not spaces)

Sorry.. :(

#8 Updated by Wael Nasreddine over 7 years ago

Damn IDEs :(... voilà, the correctly indented file, saved you 2 minutes...

P.S: It would be interesting if I could remove my own attachments...

#9 Updated by Felix Schäfer over 7 years ago

  • Target version set to 1.0.0 (RC)

Tagged for BugMash-10.

#10 Updated by Eric Davis over 7 years ago

  • Status changed from New to 7
  • Target version deleted (1.0.0 (RC))
  • % Done changed from 50 to 0

I'm getting some test failures with this patch. Can you update to the latest Redmine trunk, reapply your patch, and try running the MailHandler's tests?

  1) Failure:
test_add_issue_note(MailHandlerTest) [test/unit/mail_handler_test.rb:253]:
<false> is not true.

  2) Failure:
test_add_issue_note_should_send_email_notification(MailHandlerTest) [test/unit/mail_handler_test.rb:277]:
<false> is not true.

  3) Failure:
test_add_issue_note_with_attribute_changes(MailHandlerTest) [test/unit/mail_handler_test.rb:262]:
<false> is not true.

#11 Updated by Eric Davis about 7 years ago

  • Assignee deleted (Eric Davis)

#12 Updated by Jean-Philippe Lang about 7 years ago

  • Status changed from 7 to Closed
  • Target version set to 1.1.0
  • Resolution set to Fixed

Added in r4394. The default keyword is "Fixed version" but r4395 lets you use localized keywords.

Also available in: Atom PDF