crescendochu / access-survey

0 stars 0 forks source link

Bug found in testing Mobility devices can be recorded multiple times? #40

Closed crescendochu closed 3 months ago

crescendochu commented 3 months ago
image
crescendochu commented 3 months ago

Also it never reaches the end of the list

image
crescendochu commented 3 months ago

Okay I tried to identify the bug and looks like it's here https://github.com/crescendochu/access-survey/blob/d32e1e47bc8256a08aef6d66e4454155e29fb083/src/survey/SurveyForm/SurveyForm.js#L678

crescendochu commented 3 months ago

Okay I think I fixed this by changing remainingOptions.length === 0 to 1, which does not make sense But this is what we had a few commits ago @rrrrrrockpang