-
Notifications
You must be signed in to change notification settings - Fork 1
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add ability to show additional questions #18
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
…stions # Conflicts: # src/app/[locale]/incident/add/components/IncidentQuestionsLocationForm.tsx
…d set chosen radio inputs back to chosen input
…d set chosen radio inputs back to chosen input
…o feat/5-additional-questions # Conflicts: # src/components/ui/RadioGroup.tsx
I have completed everything required, but there are still a few things that need to be addressed (in other pull requests, to keep this one from becoming too large). The AssetSelect type question needs to be implemented (#7). This PR is best tested with a report for "Vuurwerkoverlast" (fireworks nuisance). |
@justiandevs Willen we iets van test automatisering toevoegen hier? Met vitest of playwright? |
Ik denk het wel. Hier wil ik alleen mee wachten totdat ook sowieso ifOneOf of ifAllOf functionaliteit is ingebouwd, dat zorgt voor de complexere situaties die goed doortest moeten worden, dan bouw ik dat in een keer. |
Todo
The
ifOneOf
andifAllOf
setting is currently "out of scope"More documentation on additional questions structure can be found here:
https://github.com/Amsterdam/signals-frontend/blob/main/docs/additional-questions.md
Closes #5