An error occurred fetching the project authors.
  1. 26 Feb, 2019 1 commit
  2. 25 Feb, 2019 2 commits
  3. 26 Jul, 2018 1 commit
  4. 07 Jun, 2018 2 commits
    • Felipe Artur's avatar
    • Sean McGivern's avatar
      Fix some N+1s when calculating notification recipients · 0206476a
      Sean McGivern authored
      First N+1: we may have loaded a user's notification settings already, but not
      have loaded their sources. Because we're iterating through, we'd potentially
      load sources that are completely unrelated, just because they belong to this
      user.
      
      Second N+1: we do a separate query for each user who could be subscribed to or
      unsubcribed from the target. It's actually more efficient in this case to get
      all subscriptions at once, as we will need to check most of them.
      
      We can fix both by the slightly unpleasant means of checking IDs manually,
      rather than object equality.
      0206476a
  5. 18 Apr, 2018 1 commit
  6. 26 Mar, 2018 1 commit
  7. 19 Mar, 2018 1 commit
  8. 22 Feb, 2018 1 commit
  9. 17 Jan, 2018 1 commit
    • Mario de la Ossa's avatar
      Initial work to add notification reason to emails · 23a20c20
      Mario de la Ossa authored
      Adds `#build_notification_recipients` to `NotificationRecipientService`
      that returns the `NotificationRecipient` objects in order to be able to
      access the new attribute `reason`.
      
      This new attribute is used in the different notifier methods in order to
      add the reason as a header: `X-GitLab-NotificationReason`.
      
      Only the reason with the most priority gets sent.
      23a20c20
  10. 11 Jan, 2018 1 commit
  11. 21 Aug, 2017 1 commit
    • http://jneen.net/'s avatar
      don't rely on order of notification levels · 3676275a
      http://jneen.net/ authored
      factor out #suitable_notification_level? and check manually by
      notification level. this makes the notification logic clear and actually
      reflect what is in the documentation as to what should happen with each
      setting.
      3676275a
  12. 11 Aug, 2017 2 commits
  13. 03 Aug, 2017 9 commits