You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
NOTE: This ticket was previously a list of what we need to do to retrofit the current search pages with React Hook Form. This was not going to be possible to iteratively replace elements due to the current search controls.
This meta ticket is to roll up all the tasks to implement the Basic Search Form. Additionally, the initial setup of the Search Form will set the ground work for implementing the Advanced Search Form. This will be a ground up build of the form. None of the search components or atomic components they reference should be used. (Unless they are 100% free of Redux and side effects) However, the low-level markup (input, label, etc) used to render the controls and the validation logic should be leveraged from the existing search controls.
In order to start on this all the Basic Search Integration tests need to be commented out. They will then be slowly uncommented as the code is updated so they may pass. @kate-mashkina please add any additional information here.
NOTE: This should be done in a feature branch. in creating the form, this allows the individual components to be worked on separately and just get registered when added to the form
This meta ticket is to roll up all the tasks to implement the Basic Search Form. Additionally, the initial setup of the Search Form will set the ground work for implementing the Advanced Search Form. This will be a ground up build of the form. None of the search components or atomic components they reference should be used. (Unless they are 100% free of Redux and side effects) However, the low-level markup (input, label, etc) used to render the controls and the validation logic should be leveraged from the existing search controls.
In order to start on this all the Basic Search Integration tests need to be commented out. They will then be slowly uncommented as the code is updated so they may pass. @kate-mashkina please add any additional information here.
Initial Form Setup
The text was updated successfully, but these errors were encountered: