chore: Implement webchat pre-engagement form - render forms [CHI-3696]#3897
chore: Implement webchat pre-engagement form - render forms [CHI-3696]#3897
Conversation
This reverts commit 395bb62.
stephenhand
left a comment
There was a problem hiding this comment.
One design question. Also, we should add some tests
| "react": "^17.0.2", | ||
| "react-app-rewired": "^2.2.1", | ||
| "react-dom": "^17.0.2", | ||
| "react-hook-form": "^6.15.8", |
There was a problem hiding this comment.
Since our long term aim is to move away from using this, is there any chance we could avoid adding it to a new project?
There was a problem hiding this comment.
I honestly don't know why that would be our long term goal. As I've said when this was discussed, we would have to implement and maintain all the validation rules, handle dirty states that are used in some components, among others features that we rely on. I don't think removing rhf is a good idea as I don't see the benefit. However, if we end up removing it, I would do so in a separate refactor PR if you don't mind, as I'd rather avoid having to write all the validations and form state management that are required for the components introduced in this PR.
Description
This PR implements the rendering of pre engagement form, based on the form definitions introduced in #3865.
Checklist
Other Related Issues
None
Verification steps
AFTER YOU MERGE
You are responsible for ensuring the above steps are completed. If you move a ticket into QA without advising what version to test, the QA team will assume the latest tag has the changes. If it does not, the following confusion is on you! :-P