diff options
| author | Dmitry Vyukov <dvyukov@google.com> | 2018-01-15 15:26:30 +0100 |
|---|---|---|
| committer | Dmitry Vyukov <dvyukov@google.com> | 2018-01-15 15:26:30 +0100 |
| commit | 46d78f4deb682ac280e65fbc8d7d37f4f23d083e (patch) | |
| tree | b59f31cec834e5ff8d99f38d8b17fa1548690a74 /dashboard | |
| parent | 66d492a6c7bb8f9eea386e016b95a0152004bf7c (diff) | |
dashboard/app: don't warn about emails on closed bugs
Diffstat (limited to 'dashboard')
| -rw-r--r-- | dashboard/app/reporting.go | 7 |
1 files changed, 6 insertions, 1 deletions
diff --git a/dashboard/app/reporting.go b/dashboard/app/reporting.go index 5c694cdc6..675138c3e 100644 --- a/dashboard/app/reporting.go +++ b/dashboard/app/reporting.go @@ -396,7 +396,12 @@ func incomingCommandTx(c context.Context, now time.Time, cmd *dashapi.BugUpdate, // but this is confusing and non-actionable for users. // So now we fail the update, but give empty reason, // which means "don't notify user". - log.Warningf(c, "Dup bug is already closed") + if cmd.Status == dashapi.BugStatusUpdate { + // This happens when people discuss old bugs. + log.Infof(c, "Dup bug is already closed") + } else { + log.Warningf(c, "Dup bug is already closed") + } return false, "", nil } case BugStatusFixed, BugStatusInvalid: |