lrperlmu / emotional_clarity

Micro interventions to help teens get through stress.
0 stars 2 forks source link

Fix phq merge #115

Open lrperlmu opened 4 years ago

lrperlmu commented 4 years ago

After merging into master, failing phq screen doesn't remove all the frames. Fix.

lrperlmu commented 4 years ago

Problem is that we started deleting at the wrong place because the test case has 2 blocker frames in a row. This is because the creation of the frames list is messy due to too many config flags. Merge config flags to simplify.

lrperlmu commented 4 years ago

did a quick bug fix that repairs the issue. keeping this ticket to resolve the technical debt.