Make email notifications for accepted invitations pluggable - Githubissues

OpenTreeMap / otm-core

OpenTreeMap is a collaborative platform for crowdsourced tree inventory, ecosystem services calculations, urban forestry analysis, and community engagement.

www.opentreemap.org

Other

188 stars 87 forks source link

Make email notifications for accepted invitations pluggable #3185

Closed jwalgran closed 7 years ago

jwalgran commented 7 years ago

We have decided that instead of notifying every user with management access, which could be a long list of people uninterested or confused by the notification we will instead notify only the person who sent the invitation. Additional recipients can be added by implementing a new plugin function.

Testing

Tail the syslog so that you can see the emails being sent.
Create an instance.
Invite a user and make them an administrator with management access.
Invite a third user.
Accept the invitation sent to the fourth user. Verify that the notification email was sent to just the instance owner.

Related to https://github.com/OpenTreeMap/otm-addons/pull/1544 Related to https://github.com/OpenTreeMap/otm-cloud/pull/403

Connects to https://github.com/OpenTreeMap/otm-clients/issues/388

coveralls commented 7 years ago

Coverage increased (+0.0007%) to 83.729% when pulling d79325215562a411b4a45213a7f89eac57650fd7 on jwalgran:jcw/pluggable-invite-notifications into 561bd3c61510cd926f76b5fdf57c1a811d84bfcb on OpenTreeMap:develop.