Defend against missing national aggregation entries

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

Defend against missing national aggregation entries #902

Closed mbarton closed 6 months ago

mbarton commented 6 months ago

Fixes #900 and adds logging to investigate #899

Move the copy pasted percentage calculation code into a function which we then call at all levels. Log if either the numerator or denominator are missing from an aggregation row (#899)

mbarton commented 6 months ago

As this passed CI and will be used to investigate an issue in staging, I am going to merge this without a review.

If anyone sees anything to modify please do let me know and I'll raise an additional PR :)