#6305 Merge email notifications when possible

v1.2.0
closed
nobody
General
Cory Johns
2015-08-20
2013-05-30
Chris Tsai
No

[forge:feature-requests:#117]

When more than one change is done to a ticket at the same time, e.g. changing the ticket status and posting a comment, I would like these changes to be sent in a single email notification rather than as two separate emails.

Update: This applies not only to the email notifications; discussion posts should also be merged. Look for example at the first two posts for ticket [forge:feature-requests:#97]. They belong together and should have been merged into a single entry.

I agree 100%

Discussion

  • Dave Brondsema

    Dave Brondsema - 2014-02-07
    • Labels: support, p3 --> p3, support, 42cc
    • Status: open --> in-progress
     
  • Igor Bondarenko - 2014-02-17

    Is it okay to merge those comments together (thus notifications would be merged automatically). Or do you want some logic that would merge only email notifications and would not touch comments at all?

     
  • Dave Brondsema

    Dave Brondsema - 2014-02-17

    In theory merging the comments together sounds great. But thinking more about it, I remember that we are now tracking an is_meta flag to track metadata changes separately (currently used for different activity stream handling, future possibilities for hiding metadata changes or different permissions for editing metadata comments). So, I think it would be best to keep them as separate comments. I hope with that constraint, that its still practical to merge the notifications that are created.

     
  • Igor Bondarenko - 2014-02-28
    • status: in-progress --> code-review
     
  • Igor Bondarenko - 2014-02-28

    Closed #535. je/42cc_6305

     
  • Cory Johns - 2014-03-04
    • QA: Cory Johns
     
  • Cory Johns - 2014-03-04
    • status: code-review --> closed
    • Milestone: limbo --> forge-mar-7
     
  • Dave Brondsema

    Dave Brondsema - 2015-01-05
    • Milestone: unreleased --> asf_release_1.2.0
     

Log in to post a comment.