Skip to content

Commit dd65687

Browse files
authored
Merge pull request #2826 from PolicyEngine/fix/dont-require-error
Make error_message optional in reports
2 parents 8c82ab3 + 8e5ec93 commit dd65687

File tree

2 files changed

+4
-4
lines changed

2 files changed

+4
-4
lines changed

changelog_entry.yaml

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,4 @@
1+
- bump: patch
2+
changes:
3+
changed:
4+
- Make error_message optional within report service

policyengine_api/routes/report_output_routes.py

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -157,10 +157,6 @@ def update_report_output(country_id: str) -> Response:
157157
if status == "complete" and output is None:
158158
raise BadRequest("output is required when status is 'complete'")
159159

160-
# Validate that error status has error_message
161-
if status == "error" and error_message is None:
162-
raise BadRequest("error_message is required when status is 'error'")
163-
164160
try:
165161
# First check if the report output exists
166162
existing_report = report_output_service.get_report_output(report_id)

0 commit comments

Comments
 (0)