Closed plocket closed 2 years ago
From @BryceStevenWilley (in PR comment):
I think I missed this convo when it happened: https://github.com/SuffolkLITLab/docassemble-assemblylinewizard/issues/153#issuecomment-774731126. IMO we would want this to not be in the umbrella interview, especially if we want to see how many people are using a specific form. For example, if we made an optional indigency form that was used in several other forms, X and Y, we'd want to know how many people actually used the indigency form, not just how many filled out X and Y, right?
I think we need to coordinate with the view stats package and figure out how to capture both universal and specific interview stats at different keys in the database.
Because without any prefix the stats are saved by the most top level interview file, I think we probably want this in the main code block.
We would need a different design to save stats that reflected which extra sub interviews were included in the parent interview
I think the way for us to think about this is more flexibility. Making it easier, perhaps with a YAML list of field names, for the author to specify what variables get stored in the snapshot.
We also want to think of two classes of information:
Closing this one--#255 better captures what we want, as does https://github.com/SuffolkLITLab/docassemble-InterviewStats/issues/13
store_variables_snapshot
:Originally posted by @plocket in https://github.com/SuffolkLITLab/docassemble-assemblylinewizard/issues/153#issuecomment-774731126