Skip to content

Commit

Permalink
Adding check for engagement setting on send email (#2125)
Browse files Browse the repository at this point in the history
* Changes to show all survey results to superusers

* removing hard coded values

* fixing linting

* splitting to seperate end points

* fixing auth check

* fixing linting

* merging method in service

* Handle no data error for graphs

* adding new nodata component

* adding new email for submission response

* fixing linting and testing

* Adding check for engagement setting on send email
  • Loading branch information
VineetBala-AOT authored Sep 5, 2023
1 parent c7fa0d4 commit 0a001d4
Showing 1 changed file with 3 additions and 2 deletions.
5 changes: 3 additions & 2 deletions met-api/src/met_api/services/submission_service.py
Original file line number Diff line number Diff line change
Expand Up @@ -97,8 +97,9 @@ def create(cls, token, submission: SubmissionSchema):

engagement_settings: EngagementSettingsModel =\
EngagementSettingsModel.find_by_id(engagement_id)
if engagement_settings.send_report:
SubmissionService._send_submission_response_email(participant_id, engagement_id)
if engagement_settings:
if engagement_settings.send_report:
SubmissionService._send_submission_response_email(participant_id, engagement_id)
return submission_result

@classmethod
Expand Down

0 comments on commit 0a001d4

Please sign in to comment.