[TAN-3608] Support Matrix question in form builder #10127
Conversation
amanda-anderson
changed the base branch from
master
to
fe-ranking-survey-question-feature
|
…bDotCo/citizenlab into TAN-3608-matrix-q-form-builder
amanda-anderson
changed the title
| reset(defaultValues); | ||
| } | ||
| }, [defaultValues, reset]); | ||
|
|
There was a problem hiding this comment.
So the issue we discussed actually wasn't solved by the BE in the end, so I revisited it again.
I've spent almost 2 full days on this now, and this is the only thing I've come up with which solves the issue. It causes a single extra re-render when the form is loaded if there are matrix questions, but otherwise I don't think it causes too much of a problem.
I'd still like to try and dig deeper into this issue to get it properly solved, but I'm thinking maybe for now to leave this in so I can finish up the other back office things (survey results, sense-making, etc.) and then re-visit this again next week? 🤔
amanda-anderson
changed the title
@luucvanderzee That's correct! This PR is just for support in the back office/form builder. I have another PR for the front office to make PRs smaller & code reviewing simpler :) |
amanda-anderson
changed the base branch from
fe-ranking-survey-question-feature
to
master
Supports the new matrix question type in the back office form builder.
@luucvanderzee you only need to review the FE code in this PR :)