department-of-veterans-affairs / vets-website

Frontend for VA.gov
Creative Commons Zero v1.0 Universal
242 stars 126 forks source link

fix depends for array builder #33108

Open rhasselle-oddball opened 1 week ago

rhasselle-oddball commented 1 week ago

Summary

depends: (formData, index) => ... 

Related issue(s)

Testing done

Screenshots

n/a

What areas of the site does it impact?

array builder depends.

Acceptance criteria

Quality Assurance & Testing

Error Handling

Authentication

Requested Feedback

(OPTIONAL) What should the reviewers know in addition to the above. Is there anything specific you wish the reviewer to assist with. Do you have any concerns with this PR, why?

williamphelps13 commented 6 days ago

I switched out the onNavForward and onNavBack conditional page solution in this file, for the depends solution locally and it is working nicely!

The issue I am still encountering is that on edit if the page becomes no longer shown the data is still present. See this in the following video when "wrist fracture" triggers the sideOfBody question but then on edit when "wrist fracture" is updated to "asthma" which does not trigger the sideOfBody the sideOfBody data is still present and "asthma" is displayed as "asthma, right":

https://github.com/user-attachments/assets/0162d64d-f227-46dc-8559-d46300b693f6

Could be out of scope for this fix or could be I need to handle it in a different way.

rhasselle-oddball commented 6 days ago

Thanks @Midge-dev for helping with the tests on 686

Status update: Working on bug that @williamphelps13 found. Need to fix that before merging.

rhasselle-oddball commented 1 day ago

Previous bug: when dependent page is the last page and its the last item added the user could not continue on from the list loop This is now resolved.

Potential new issue: the recent changes appear to have changed formData to contain all formData for onNavForward when it previously only contained the itemData.

This may be an enhancement actually because it seems desirable that formData always contain all formData and never just contain itemData.

Appreciate the thorough testing @williamphelps13 - I'll spend some more time on this, to make sure the formData discrepancies don't cause regressions in existing forms.