Small comment mail fix #192

Merged
matthijskooijman merged 3 commits from small-comment-mail-fix into master 2019-01-17 20:06:44 +00:00
matthijskooijman commented 2019-01-17 19:18:44 +00:00 (Migrated from github.com)

This prevents sending public comment notifications for review comments, which was caused by an older bug and exposed by #183. I've also included some code to generalize loading a comment author. This is probably not really required anymore with the fix in the last commit, but doesn't hurt either.

This prevents sending public comment notifications for review comments, which was caused by an older bug and exposed by #183. I've also included some code to generalize loading a comment author. This is probably not really required anymore with the fix in the last commit, but doesn't hurt either.
Sign in to join this conversation.
No reviewers
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
harmen/hypha!192
No description provided.