wfu-dmds / teaching-r-study

3 stars 2 forks source link

Update consent, demographics, and learnr module to incorporate user ids and deploy to #9

Closed LucyMcGowan closed 3 years ago

LucyMcGowan commented 3 years ago

@jdtrat: copying this over from our email

With the new changes, can you please update the learnr study by:

jdtrat commented 3 years ago

@LucyMcGowan Should I use the original consent.html and home.html documents? If so, is it okay to bundle the apps in a folder with the traditional Shiny structure?

LucyMcGowan commented 3 years ago

Yes! Sorry I should have left those in the repo - do you have the locally? I can push them if not. And yes! Feel free to have them in their own folders

jdtrat commented 3 years ago

No worries! I'll grab them from the GitHub history.

jdtrat commented 3 years ago

Please see latest commit. All are addressed and when tested was functional. I hope to clean up the code but wanted to get this to you ASAP. You can walk-through it yourself here.

LucyMcGowan commented 3 years ago

This looks great! The only thing I didn’t see added were inputs for the likert scale questions for the assessment. I tried it on mobile too and the survey came out a bit wonky (but this is definitely low priority, just thought I’d mention so we can track it)

0454537D-5C9B-4D02-8C76-3B0E123BFBE6

jdtrat commented 3 years ago

I'm not sure what you mean re: likert questions/assessment. For each assessment, am I supposed to have a include a shinysurvey that tracks their responses?

Also, that's a {shinysurveys} issue. The scaling doesn't work properly. If you have a chance, I'd appreciate your taking a look at the CSS file!

jdtrat commented 3 years ago

I'm not sure what you mean re: likert questions/assessment. For each assessment, am I supposed to have a include a shinysurvey that tracks their responses?

Also, that's a {shinysurveys} issue. The scaling doesn't work properly. If you have a chance, I'd appreciate your taking a look at the CSS file!

The sass file is here and you can see a demo (for browser inspection) here.

LucyMcGowan commented 3 years ago

Ah I meant just adding a shiny element that will save to the same Dropbox folder with their username for the first 3 questions in assessment.Rmd (right now the questions are there but no way to input responses)

jdtrat commented 3 years ago

Should the same exercises appear on each assessment or just the questions?

LucyMcGowan commented 3 years ago

Yes! All 3 assessments are exactly the same

jdtrat commented 3 years ago

Just added that assessment functionality. You can see the files in the Dropbox. We should do some more testing, but it seems to work fine!

jdtrat commented 3 years ago

This looks great! The only thing I didn’t see added were inputs for the likert scale questions for the assessment. I tried it on mobile too and the survey came out a bit wonky (but this is definitely low priority, just thought I’d mention so we can track it)

0454537D-5C9B-4D02-8C76-3B0E123BFBE6

This was fixed in the latest update to {shinysurveys} and I've redeployed the survey-app!