Skip to content

Resolve "C_004_R8 Notes Progress Cue Count"

Noel Simmel requested to merge 1228-c_004_r8-notes-progress-cue-count into main

Closes #1228 (closed)

After talking to Elisabeth again, I decided to just exclude the daily notes question directly via its ID.

Reasoning:

  1. We don't foresee that we want this behavior for other questions. (The reason we want to exclude the daily notes in the first place is because it's "forced" in all symptom checks by the app, not chosen by the HCP.)
  2. Excluding all "forced" questions on a lower level is possible in theory, but would involve looking at a mechanism from the HCP side (that's where the daily notes are added to the symptom check). This violates separation of concerns, as it mashes up PAT and HCP logic.
  3. This approach is in line with how daily notes are currently treated in other parts of the app: E.g. the PDF service differentiates between daily notes (via ID) and all other questions. Or when saving templates, we also explicitly look for the daily notes ID (isDailyNotesQuestion()) and exclude it. (Of course, if that is the best approach in general is up for debate, but I currently see no reason to change it, see point 1)

Merge request reports