search

stats4sd / aec_portfolio

A proof of concept for the AEC Consortium Project Management / Assessment System
GNU General Public License v3.0
0 stars 0 forks source link

Enhance Dashboard #122

Closed dan-tang-ssd closed 1 year ago

dan-tang-ssd commented 1 year ago

In principle summary, all principles assessed in latest assessments are included even assessment.redline_status is not Complete.

This is not common, but it could happen for below scenario: After principle assessment, user can assess red flag and change any red flag to YES to have 0 score.

This PR is submitted to exclude any latest assessments if it's redline_status is not Complete.

Screen shots:

A portfolio has two initiative. The first initiative has failed red flags assessment.

image

BEFORE: Last assessments of both two initiatives are included in principle summary. image

AFTER: The first failed initiative is excluded from principle summary. Only the second Complete assessment is included in principle summary. image

dan-tang-ssd commented 1 year ago

Live env deployment is completed. Performed testing with positive result.