str -> RequestStatus coercion is no longer necessary

This commit is contained in:
richard-dds 2018-08-07 17:07:25 -04:00
parent 5381e68a8d
commit e437b0d6ba

View File

@ -117,16 +117,11 @@ class Requests(object):
@classmethod @classmethod
def action_required_by(cls, request): def action_required_by(cls, request):
try:
status = RequestStatus(request.status)
except ValueError:
return None
return { return {
RequestStatus.STARTED: "mission_owner", RequestStatus.STARTED: "mission_owner",
RequestStatus.PENDING_FINANCIAL_VERIFICATION: "mission_owner", RequestStatus.PENDING_FINANCIAL_VERIFICATION: "mission_owner",
RequestStatus.PENDING_CCPO_APPROVAL: "ccpo" RequestStatus.PENDING_CCPO_APPROVAL: "ccpo"
}.get(status) }.get(request.status)
@classmethod @classmethod
def should_auto_approve(cls, request): def should_auto_approve(cls, request):