-
Notifications
You must be signed in to change notification settings - Fork 483
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
AYW: Update content for post survey #36408
Conversation
"text": "(virtual meeting) Discussing how to counsel students who are disengaging or becoming more passive in class." | ||
} | ||
] | ||
"html": "<h2>How useful was Module 1?</h2>\n\n<p><span style=\"font-family: arial, helvetica, sans-serif;\">For each activity in this module, we ask that you <strong>rate how useful that activity was to you <em>personally</em></strong> in terms of your own <strong>learning, development, teaching practice</strong>, etc. </span></p>" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
should the "module_1" matrix be removed?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Do you mean did I intentionally remove that set of questions? I think the answer is yes, let me confirm with Laura though. She also removed the same set of questions for module 2, I believe.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ah ok I was slightly confused because it looked like you kept the prompt but removed the question, but it looks like prompt actually goes with the next question as well
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yep, I think that's right.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM!
Updated content for AYW post surveys. Took a while to figure out how to reproduce the diffs generated in Laura's new version of the survey in the library questions that generated the survey (even though she told me what they were, and, as you can see in this PR, they're not huge).
Testing story
I made appropriate changes to library question, then re-seeded library questions, then rendered the JSON of the form via /foorm/editor. I used an online JSON diff tool to compare the original JSON file that Laura had provided me with the one generated when using the library questions as updated in this PR and saw no diffs.
Reviewer Checklist: