Skip to content

Commit 17d85a6

Browse files
authored
Merge pull request #121 from ModusCreateOrg/ADE-66
[ADE-66] update report processing logic to handle isMedicalReport correctly and improve error handling
2 parents 6eac454 + f365669 commit 17d85a6

File tree

2 files changed

+9
-5
lines changed

2 files changed

+9
-5
lines changed

backend/src/reports/reports.service.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -422,7 +422,7 @@ export class ReportsService {
422422
{ field: 'confidence', value: report.confidence },
423423
{ field: 'status', value: report.status },
424424
{ field: 'missingInformation', value: report.missingInformation || [] },
425-
{ field: 'isMedicalReport', value: report.isMedicalReport || false },
425+
{ field: 'isMedicalReport', value: report.isMedicalReport || undefined },
426426
{ field: 'errorMessage', value: report.errorMessage || '' },
427427
{ field: 'updatedAt', value: report.updatedAt },
428428
];

frontend/src/pages/Processing/ProcessingPage.tsx

Lines changed: 8 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -72,11 +72,15 @@ const ProcessingPage: React.FC = () => {
7272
console.log('Processing complete');
7373

7474
history.push(`/tabs/reports/${reportId}`);
75-
} else if (data.isMedicalReport === false) {
76-
setIsProcessing(false);
77-
clearStatusCheckInterval();
78-
setError(missingDataHeading, missingDataMessage);
7975
} else if (data.status === 'failed') {
76+
if (data.isMedicalReport === false) {
77+
setIsProcessing(false);
78+
clearStatusCheckInterval();
79+
setError(missingDataHeading, missingDataMessage);
80+
81+
return;
82+
}
83+
8084
throw new Error();
8185
}
8286
} catch {

0 commit comments

Comments
 (0)