Feature/arc 3281 approve deny requests

[reunefe]

Continues on #573

[bertyhell]

usually i make a mapping object:

const MAP_MATERIAL_REQUEST_STATUS_TO_EMAIL_TEMPLATE = {
	[Lookup_App_Material_Request_Status_Enum.Cancelled]: EmailTemplate.MATERIAL_REQUEST_REQUESTER_CANCELLED,
	[Lookup_App_Material_Request_Status_Enum.Approved]: EmailTemplate.MATERIAL_REQUEST_MAINTAINER_APPROVED ,
	[Lookup_App_Material_Request_Status_Enum.Denied]: EmailTemplate.MATERIAL_REQUEST_MAINTAINER_DENIED,
}

and then you can use is more easily:

const emailTemplateToSend: EmailTemplate | undefined = MAP_MATERIAL_REQUEST_STATUS_TO_EMAIL_TEMPLATE[send] || undefined;

this makes it easier to reuse in other places

[bertyhell]

try to avoid parameters in the message
put all variables in the additional info at the end
that way, if we capture this error somewhere higher up in the callstack, we can easily use the additional info to choose how to deal with this error

Suggested change

	`Failed to send email to maintainer that the material request was ${request.status}`,
	err,
	{ request, user }
	'Failed to send email about material request status update',
	err,
	{ status: request.status, template, request, user }