Conversation
…ceText and sampleText for addQuestionCustomization and added a new findByCustomizationAndVersionedQuestion function for QuestionCustomization model
…n has sample text when it is a blank string
| if [ -f "/etc/mysql-cert.pem" ]; then | ||
| MIGRATION_ARGS="${MIGRATION_ARGS} --ssl-ca=/etc/mysql-cert.pem --ssl-verify-server-cert=OFF" | ||
| fi | ||
| MIGRATION_ARGS="${MIGRATION_ARGS} --ssl=off" |
There was a problem hiding this comment.
The -ssl-verify-server-cert=OFF" was not working for me and the nuke-db.sh was breaking so I switched to using --ssl=off for both this and process.sh and it worked
There was a problem hiding this comment.
Can you leave the --ssl-verify-server-cert=OFF commented out below the line?
I'm not sure why our mariadb would be different, but I'd prefer to keep that there in case I need to use it on mine
briri
left a comment
briri
left a comment
There was a problem hiding this comment.
Looks good @jupiter007 Please just leave that ssl argument commented out below for me in case we need to use it. I'm not sure why our local machines would treat it differently.
| if [ -f "/etc/mysql-cert.pem" ]; then | ||
| MIGRATION_ARGS="${MIGRATION_ARGS} --ssl-ca=/etc/mysql-cert.pem --ssl-verify-server-cert=OFF" | ||
| fi | ||
| MIGRATION_ARGS="${MIGRATION_ARGS} --ssl=off" |
There was a problem hiding this comment.
Can you leave the --ssl-verify-server-cert=OFF commented out below the line?
I'm not sure why our mariadb would be different, but I'd prefer to keep that there in case I need to use it on mine
2 participants
Description
guidanceTextandsampleTextfields toaddQuestionCustomizationand addedjson,questionText,requirementText,guidanceText,sampleText,useSampleTextAsDefaultandrequiredtoaddCustomQuestionInputquestionCustomizationByVersionedQuestionresolverfindByCustomizationAndVersionedQuestionmethod toQuestionCustomizationmodelfetchTemplateDataquery inTemplateCustomizationmodel becausequestionCustomizationHasSampleTextwas incorrectly returning trueFixes # (130)
Type of change
How Has This Been Tested?
Manually tested with frontend and also with new unit tests
Checklist: