adding_required_fields_logic

[aranas]

This is adding logic to check for required fields. The required fields are specified in the config file.

Currently there is still an issue with open text fields, somehow the logic only works for some of them.

[aranas]

no it is a good point, this is probably why it is buggy with the open text questions! Will have to check that

[aranas]

I just pushed a new commit that fixes the required field logic such that

• fields are checked after every page (upon button click "continue")
• required fields are checked for input taking into account type list for multiselect widgets
• required fields are checked for default setting "Select"

Note that for sliders, the user will have to move the slider for the response to be logged!! this might be confusing

[aranas]

@cptanalatriste the required field logic works now but the widget type specific tests are hardcoded, so not particularly elegant. We will need to keep an eye on this as we go updating the app. I'll add it to the technical test

Would you like to have a look before I merge?

[aranas]

@cptanalatriste could you review just the 6d341af commit, specifically the changes I've made to streamlit_utils.py. I have wrapped the generate_streamlit_element into a class to keep track of number.

I want to make sure this doesn't interfere with other bits of the code in a way that I am not aware of