Closed Bug 1397527 Opened 7 years ago Closed 7 years ago

thNotify.send shouldn't persist most "success" messages.

Categories

(Tree Management :: Treeherder, enhancement, P1)

Product:
Tree Management
Component:
Treeherder
Type:
enhancement

Tracking

(Not tracked)

Status:
RESOLVED FIXED

People

(Reporter: KWierso, Assigned: KWierso)

References

Details

Attachments

(1 file)

[treeherder] KWierso:dontpersist > mozilla:master
(deleted), text/x-github-pull-request
camd
: review+
Details
Bug 1391321 redid a few thNotify.send() calls, making a few of the success cases persist where they previously didn't persist (probably just a copy/paste of instances found elsewhere?).
Assignee: nobody → wkocher
Blocks: 1391321
Status: NEW → ASSIGNED
Priority: -- → P1
We did this because we get bug reports where "something didn't work" and no way to know which task was created. I guess it doesn't hurt too much to remove it again if it is causing annoyance. I'll try to work on some way to make both parties happy eventually.
Attachment #8905301 - Flags: review?(cdawson) → review+
Status: ASSIGNED → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: