Project

General

Profile

Actions

Feature #5622

closed

Allow version to be set via incoming email

Added by Wael Nasreddine over 13 years ago. Updated over 13 years ago.

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

0%

Estimated time:
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


Files


Related issues

Related to Redmine - Feature #5594: Improve MailHandler's keyword handlingClosed2010-05-25

Actions
Actions #1

Updated by Alexsandro Farias over 13 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

Actions #2

Updated by Felix Schäfer over 13 years ago

  • Status changed from Resolved to New
Actions #3

Updated by Wael Nasreddine over 13 years ago

Ummmm Guys!! What's going on?

Actions #4

Updated by Felix Schäfer over 13 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.

Actions #5

Updated by Wael Nasreddine over 13 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 :)

Actions #6

Updated by Wael Nasreddine over 13 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...

Actions #7

Updated by Wael Nasreddine over 13 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.. :(

Actions #8

Updated by Wael Nasreddine over 13 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...

Actions #9

Updated by Felix Schäfer over 13 years ago

  • Target version set to 1.0.0 (RC)

Tagged for BugMash-10.

Actions #10

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

Actions #11

Updated by Eric Davis over 13 years ago

  • Assignee deleted (Eric Davis)
Actions #12

Updated by Jean-Philippe Lang over 13 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.

Actions

Also available in: Atom PDF