search

rcpch / rcpch-audit-engine

Epilepsy12 Audit Platform
https://e12.rcpch.ac.uk/
GNU Affero General Public License v3.0
5 stars 5 forks source link

optional-input-dates #1097

Closed eatyourpeas closed 2 weeks ago

eatyourpeas commented 3 weeks ago

Overview

This makes the input dates for paediatrician with expertise, neurologist and epilepsy specialist nurse all optional, and introduces input_achieved toggles that make them mandatory if selected.

The E12 team do not feel this has implications for the KPI calculations

Generates 3 new fields in the Assessment model so will there is a new migration

closes #1095

eatyourpeas commented 3 weeks ago

just to add, it adds a final migration that retrospectively sets the *_input_achieved value to True for those existing records where *_input_date is not None. Screen shots of the changes are in the issue it closes.

eatyourpeas commented 3 weeks ago

@mbarton just because of the extra migration step at the end would be grateful if you could try out that bit and make sure it does what we expect - ie all existing records where epilepsy_specialist_nurse_input_date, consultant_paediatrician_input_date or paediatric_neurologist_input_date in the Assessment tab have existing dates, that they are visible in the ui and the *_input_achieved toggle is set to true

mbarton commented 2 weeks ago

Seen on STAGING (merged by @eatyourpeas 28 minutes and 31 seconds ago) Please check your changes!

mbarton commented 2 weeks ago

Seen on LIVE (merged by @eatyourpeas 28 minutes and 35 seconds ago) Please check your changes!