Homec4science

Projects - add mail to project updates

Authored by Bob Trahan <btrahan@phacility.com> on May 16 2015, 01:33.

Description

Projects - add mail to project updates

Summary:
...which lets all the fancy settings for Email | Notify | Off be possible. Fixes T8164. Wasn't too sure the best way to break things up but members vs watchers felt meaningful to break out to me.

Also fixes a small bug where we were generating bad slug updated stories by messing with the signature of the slug data. Perhaps this fix isn't even good enough (the array_keys()) call and instead we'll need to implement transaction has effect and do a sort?

Test Plan: used ./bin/mail list-outbound and ./bin/mail show-outbound --id XX to verify reasonable emails were being generated. saw new preferences in settings.

Reviewers: epriestley

Reviewed By: epriestley

Subscribers: Korvin, epriestley

Maniphest Tasks: T8164

Differential Revision: https://secure.phabricator.com/D12868

Details

Committed
Bob Trahan <btrahan@phacility.com>May 16 2015, 01:33
Pushed
aubortJan 31 2017, 17:16
Parents
rPH3ef0721adacc: Reduce PhabricatorUser::getOmnipotentUser calls by adding a getViewer method to…
Branches
Unknown
Tags
Unknown

Event Timeline

Bob Trahan <btrahan@phacility.com> committed rPHfa82c17079de: Projects - add mail to project updates (authored by Bob Trahan <btrahan@phacility.com>).May 16 2015, 01:33