When a moderator reports another user, they should not receive a notification email

nlhkabu / connect

Connect is a moderated web application that helps members connect with each other based on skills, interests and location.

http://whoisnicoleharris.com/connect/

BSD 3-Clause "New" or "Revised" License

46 stars 25 forks source link

When a moderator reports another user, they should not receive a notification email #4

Closed Natim closed 9 years ago

nlhkabu commented 9 years ago

Probably a good idea to check all the places in the codebase where moderators are receiving emails to check that they're not getting anything they're not supposed to. Will need to improve unit tests to cover this too.

nlhkabu commented 9 years ago

This is only an issue when the moderator reports another user, not when they are being reporting by another user.

nlhkabu commented 9 years ago

Fixed with 5760ff71bd244f9043d57545429318b62d598713

Natim commented 9 years ago

Can you point me in the commit above the line that test this behavior please?

nlhkabu commented 9 years ago

Whoops. Wrong commit. Here it is @Natim: Line https://github.com/nlhkabu/connect/commit/8081ff92724c6a3b8b87bdcc1824a28a788b9011#diff-b190009fdce27889e3b62d2d924ae4c1R642