sssoja commented 2 years ago

What changes have you made?

Refactored form.js and form.test.js to be a duplicate of the code in replybot
Added in any extra unit tests that do not currently exist in replybot
Refactored validator.js and validator.test.js to be more aligned to the needs of the websurvey
Updated responsStore.js and responseStore.test.js in line with these changes
Removed any instances of field type legal as we've since decided to deprecate this feature
Merged #93 into this branch so that it includes components of additional field types opinion_scale and rating

Which issue(s) does this PR address?

88

How to test

Run npm install and open up the web survey on localhost:5000
Check that all tests are passing
You should be able to switch around the JSON by importing the data from test-1 and test-2 without any breaks in tests or functionality

nandanrao commented 2 years ago

LGTM!

vercel[bot] commented 2 years ago

This pull request is being automatically deployed with Vercel (learn more).
To see the status of your deployment, click below or on the icon next to each commit.

🔍 Inspect: https://vercel.com/fly-websurvey/fly/BhsDzGv9TRmXKREuFGnNCzQcLTn3
✅ Preview: https://fly-git-optimise-code-sharing-between-chat-ff4ec9-fly-websurvey.vercel.app

vlab-research / fly

Optimise code sharing between chatbot and web survey #94

What changes have you made?

Which issue(s) does this PR address?

88

How to test