Feat: conditional question logic #50
Conversation
|
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
…ay them by default.
|
Ik ga het toch op een andere manier aanpakken denk ik, met de render logica in het component zelf in plaats van in de algemene useEffect. Ik verwacht dan minder problemen tegen te komen. https://chatgpt.com/share/671ff3b1-d49c-800d-a175-0606b481fbb4, dit is wat chatgpt mij als advies gaf en dit: https://github.com/Amsterdam/signals-frontend/blob/472508e94719ebcb7be8840e893872ff4d8a5977/src/signals/incident/services/checkVisibility/index.js is de huidige Amsterdam code. Weet dus nu ongeveer in welke hoek ik het ga zoeken. |
justiandevs
changed the title
…estions # Conflicts: # src/app/[locale]/incident/add/components/IncidentQuestionsLocationForm.tsx # src/app/[locale]/incident/add/components/questions/CheckboxInput.tsx # src/app/[locale]/incident/add/components/questions/RadioInput.tsx # src/app/[locale]/incident/add/components/questions/TextAreaInput.tsx # src/app/[locale]/incident/add/components/questions/TextInput.tsx
…factor -> radiogroup
src/app/[locale]/incident/add/components/questions/TextAreaInput.test.tsx
Outdated
Show resolved
Hide resolved
src/app/[locale]/incident/add/components/IncidentQuestionsLocationForm.tsx
Outdated
Show resolved
Hide resolved
src/app/[locale]/incident/add/components/IncidentQuestionsLocationForm.tsx
Outdated
Show resolved
Hide resolved
src/app/[locale]/incident/add/components/questions/CheckboxInput.tsx
Outdated
Show resolved
Hide resolved
|
Het ziet er echt heel netjes uit. 🚀 Over het algemeen heel goed te volgen wat hier gebeurt. Ik heb nog een belangrijk structuur-ding, verder allemaal minor. |
…est for RenderSingleField
…estionsLocationForm.tsx
Fixes #6 and integrates unit tests for each input type to check if it should be rendered or not based on conditional logic.