#8157 Improvements to multiple commits in single notification

v1.8.0
closed
None
General
nobody
2018-02-05
2017-07-06
No

When pushing multiple commits, the email notification has a heading for the branch name, but it is at the bottom instead of the top of the email. This is probably broken since [#8144] reversed the order.

It would also be good to have a max # of commits included in the email, especially for cases when a full repo gets pushed and reindex with notifications on.

Related

Tickets: #8144

Discussion

  • Dave Brondsema

    Dave Brondsema - 2017-07-06
    • status: in-progress --> review
     
  • Dave Brondsema

    Dave Brondsema - 2017-07-06

    Fixed on db/8157 (also [#7896] fixes)

     

    Related

    Tickets: #7896

  • Kenton Taylor

    Kenton Taylor - 2017-07-12
    • status: review --> closed
     
  • Kenton Taylor

    Kenton Taylor - 2017-07-12

    Merged.

     
  • Dave Brondsema

    Dave Brondsema - 2018-02-05
    • Milestone: unreleased --> v1.8.0
     

Log in to post a comment.