"levelOfSupervision" field in VisitReports table gets nulled when reports are approved

[gregorp]

In the VisitReports table, the levelOfSupervision field gets nulled unexpectedly in a new version when the state changes to 'approved'. Heres a couple sample visit reports:

select id, "versionCreatedAt", "levelOfSupervision", "visitType", "state"
where id in (1585945, 1589351)
order by id, "versionCreatedAt"

This seems to have been the case since 2018.

The visitType field what is used for billing and dashboard. The levelOfSupervision field does not contradict visitType:

-- demonstrating that levelOfSupervision does not contradict visitType
select "levelOfSupervision", "visitType", * 
from replica."VisitReports" where
state <> 'approved'
AND date > '2025-01-01'
AND coalesce("levelOfSupervision", "visitType") <> "visitType"
LIMIT 500;

Some options:

1. Document the levelOfSupervision field should not be used
2. Drop the levelOfSupervision field
3. Fuller fix - straighten out the difference between supervision level (unsupervised, monitored, supervised), transport needs (yes or no) and whether it's a parent or sibling visit (also called visit type in some places)

[cbeaulaurier]

Some notes from our internal conversation:

• When a social worker fills out a visit plan in FamLink they have three options for one question “What is the level of supervision?
• And the second question is “Are you asking the provider transport the child?”
• Technically in FamLink they can even put unsupervised and no transport, which is a clue that social workers understand what they’re asking for
  • So we think social workers fill it out correctly
• We allow providers to override the referral and change the level of supervision when they’re filling out a visit report
• We tried to clamped own on this in the past and all chaos ensued – sometimes the social worker can’t get the new referral done
  • Providers want to keep doing visits but they need the referral
• And there’s a third thing: the fields for visit type in attended visit report and missed visit report are different -- and this should be straightened out if we were to open this up.