US-EPA-CAMD / easey-ui

Project Management repo for EPA Clean Air Markets Division (CAMD) Business Suite of applications
MIT License
0 stars 0 forks source link

Evaluation Report Updates #5959

Open ibarra-michelle opened 10 months ago

ibarra-michelle commented 10 months ago

The Evaluation Status Code is missing from the Monitoring Plan Evaluation Report. In contrast, the ECMPS Evaluation Report email sent to the user does include the Evaluation Status Code.

This issue can be seen in the Monitoring Plan Evaluation Reports for the Facilities below:

Acceptance Criteria

annalbrecht commented 6 months ago

The report has a severity column which is really the same as the Evaluation Status Code. Need requirements from EPA.

annalbrecht commented 6 months ago

Looked at this with Chris on 3/28. He is going to talk to Michelle about this ticket.

annalbrecht commented 5 months ago

Discussed with EPA on 4/11/24: Update MP, QA, EM evaluation reports to remove the link associated with the "Evaluation Status Code" (keep the text). Also remove lat/long info from the email and eval reports.

lgiannini1 commented 3 months ago

Verified on dev that Evaluation Status Code links are removed for all Evaluation Report email data types as well as latitude/ longitude info from Evaluation Report emails and attachments.

Passed evaluation reports do not contain any Evaluation Results. Was this ticket supposed to resolve this issue?

Image

annalbrecht commented 3 months ago

@lgiannini1 please add acceptance criteria.

mxtomoto1 commented 2 months ago

Evaluation Report reports "Evaluation has passed without errors" when there are no critical errors. When there are critical errors, Evaluation Status Code in the email reports "Critical Errors", and the Report's Evaluation Results section lists the errors out. Lat / Long info and "Evaluation Status Code"-related links do not appear.

t_d4 t_d3 t_d44

(Critical error Evaluation Report generated when tester attempted to create a new component / analyzer range without all correct information under an existing system.)

ibarra-michelle commented 2 months ago

Verified in test for ORIS 470 Location 1 and ORIS 6137 Location 1: