Allows for customizing contact information in email notifications

[mvilanova]

No description provided.

[lgtm-com]

This pull request introduces 1 alert and fixes 1 when merging 2ab61fa into 1e163bd - view on LGTM.com

new alerts:

• 1 for Potentially unsafe external link

fixed alerts:

• 1 for Potentially unsafe external link

[lgtm-com]

This pull request fixes 1 alert when merging 8cc1d8f into 1e163bd - view on LGTM.com

fixed alerts:

• 1 for Potentially unsafe external link

[kevgliss]

Just so I'm clear, this PR allows you to specify a default contact (dispatch_help) or allows you to provide a service that resolves the contact depending on when the message is sent?

Also, please modify the docs for the new configuration variable. Also since we are deprecating a configuration item we should first warn (in the docs and the code) that we are deprecating it for a bit before removing it.

I think in this case since it's just a rename we can just alias them until we remove.

[mvilanova]

It allows you to provide the contact information you want to use in the base template footer. In the case of the task reminder, we're using the current oncall (if INCIDENT_ONCALL_SERVICE_ID is set), as it seems to be the best option. For all the other cases, the incident commander, so we keep discussions contained within the incident.

[lgtm-com]

This pull request fixes 1 alert when merging a73e631 into 1e163bd - view on LGTM.com

fixed alerts:

• 1 for Potentially unsafe external link