Feature #6033
closed
Allow addition/removal of subtasks to show in parent's history
0%
Description
The subtask feature has been working as expected. When I remove a parent from a child, that change is captured in the child's history, but it does not appear in the parent's history. I think it would be really handy to be able to see from the parent's perspective, how many child issues came and went. Seeing history like this viewing from the child up to the parent is a tad cumbersome. Hopefully, that makes sense.
Bravo Zulu on Redmine.
Files
Related issues
Updated by Yuichi HARADA over 4 years ago
Display the addition / deletion of subtask issues in the parent issue's history block.
I attached a patch.
Updated by Go MAEDA over 4 years ago
- Target version set to Candidate for next major release
Updated by Go MAEDA over 4 years ago
- Subject changed from Allow removal of child subtasks to show in parent's History block to Allow addition/removal of subtasks to show in parent's History block
- Target version changed from Candidate for next major release to 4.1.0
Setting the target version to 4.1.0.
Updated by Marius BĂLTEANU about 3 years ago
- Target version changed from 4.2.0 to 5.0.0
Updated by Go MAEDA over 2 years ago
Update the patch for the latest trunk (r21204).
Updated by Go MAEDA over 2 years ago
The patch breaks an existing test. This is because patched Redmine sends an email notification when a journal that records the addition or removal of a subtask is added.
Error:
MailHandlerTest#test_add_issue_should_send_notification:
NameError: undefined local variable or method `byebug' for #<MailHandlerTest:0x00007fd3f69b1d98>
test/unit/mail_handler_test.rb:438:in `test_add_issue_should_send_notification'
rails test test/unit/mail_handler_test.rb:431
Updated by Yuichi HARADA over 2 years ago
- File 6033_subtasks_show_in_parent_history-v2.patch 6033_subtasks_show_in_parent_history-v2.patch added
Go MAEDA wrote:
The patch breaks an existing test. This is because patched Redmine sends an email notification when a journal that records the addition or removal of a subtask is added.
[...]
I have confirmed that the test fails. I fixed test/unit/mail_handler_test.rb as follows.
diff --git a/test/unit/mail_handler_test.rb b/test/unit/mail_handler_test.rb
index 3fd3ce0722..999cb8ceac 100644
--- a/test/unit/mail_handler_test.rb
+++ b/test/unit/mail_handler_test.rb
@@ -433,10 +433,19 @@ class MailHandlerTest < ActiveSupport::TestCase
assert issue.is_a?(Issue)
assert !issue.new_record?
- mail = ActionMailer::Base.deliveries.last
- assert_not_nil mail
- assert mail.subject.include?("##{issue.id}")
- assert mail.subject.include?('New ticket on a given project')
+ assert_equal 4, issue.parent_issue_id
+ assert_equal 2, ActionMailer::Base.deliveries.size
+
+ [
+ [issue.id, 'New ticket on a given project'],
+ [4, 'Issue on project 2'],
+ ].each do |issue_id, issue_subject|
+ mail =
+ ActionMailer::Base.deliveries.detect do |m|
+ /##{issue_id}/.match?(m.subject) && /#{issue_subject}/.match?(m.subject)
+ end
+ assert_not_nil mail
+ end
end
def test_created_user_should_be_added_to_groups
Updated by Go MAEDA over 2 years ago
- Subject changed from Allow addition/removal of subtasks to show in parent's History block to Allow addition/removal of subtasks to show in parent's history
- Status changed from New to Closed
- Assignee set to Go MAEDA
- Resolution set to Fixed
Committed the patch. Thank you for your contribution.
Updated by Go MAEDA over 2 years ago
- Has duplicate Feature #35991: Parent ticket history plugin added