Roundup Tracker - Issues

Issue 2550957

classification
Title: Duplicate emails (with patch)
Type: behavior Severity: normal
Components: Infrastructure Versions: devel
process
Status: fixed Resolution: fixed
Dependencies: Superseder:
Assigned To: rouilj Nosy List: rouilj, trentgg
Priority: Keywords: patch

Created on 2017-11-06 10:33 by trentgg, last changed 2017-11-08 02:02 by rouilj.

Files
File name Uploaded Description Edit Remove
roundup-2.diff trentgg, 2017-11-06 10:33 Patch for this issue.
Messages
msg6050 Author: [hidden] (trentgg) Date: 2017-11-06 10:33
I like to get notifications of all changes on my bug tracker so I added
myself to the bcc of a nosyreaction. I get 2 emails for example if I
close my own bug report. I'm attaching a patch which fixes this.

Previously not all users were added to the "seen" map. This just marks
them all as seen (ie, message sent already.)
msg6051 Author: [hidden] (rouilj) Date: 2017-11-08 02:02
Fixed in changeset: 5312:a4fe76fba54a

I also added a test that without the patch exhibits duplicate addresses.
With the patch no dupes.

Thanks for the patch Trent.

Closing.
History
Date User Action Args
2017-11-08 02:02:16rouiljsetstatus: new -> fixed
assignee: rouilj
resolution: fixed
messages: + msg6051
nosy: + rouilj
2017-11-06 10:33:27trentggcreate