-
Notifications
You must be signed in to change notification settings - Fork 34
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
Add support for Materia widgets in assessments. #1912
base: dev/33-serpierite
Are you sure you want to change the base?
Add support for Materia widgets in assessments. #1912
Conversation
…es available to the viewer component.
…s and display them in the viewer. Added Materia as a type option for questions. Added support to assessment reviews for partial scores.
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.
Super excited this is happening. I haven't tested any UI yet, just digging in the code. Biggest thing I'm seeing is that the obo Question-y flow is being circumvented, so Materia will need to be put in there somehow (elaborated in one of the comments).
I definitely could see this requiring changes at the Question viewer-component level (or hopefully not, but maybe changes to QuestionUtil or QuestionStore). Best case is not having to do that, but I don't think it's terrible if that ends up being the result.
packages/obonode/obojobo-chunks-materia/server/materiaassessment.js
Outdated
Show resolved
Hide resolved
…sessment-specific code was moved out of the Materia chunk and into a new MateriaAssessment chunk. A new table was created to store relevant external tool data payloads according to user/visit/draft content/node IDs, and this table is being used to verify Materia widget scores instead of the LTI grade passback event records. Finalized support for partial scores in question outcomes.
… component to be passed down to assessment components instead of onFormChange. onFormChange calls onSaveAnswer for form-based assessment types, non-form-based assessment types can call onSaveAnswer directly.
This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. We do this to keep our backlog under control, but we thank you for your contributions. |
…licts, adjusted migration date, added some logic to prevent Materia assessments from being a selectable option if the relevant node is not registered, made the Materia assessment node appropriately optional.
THIS BRANCH HAS A MIGRATION.
Closes #1772.
Adds Materia chunk score verification and displaying highest scores via LTI passback in the viewer.
Adds Materia as a type option for questions.
Also adds support for displaying partial credit in assessment reviews. This is sort of approaching #545, but will likely need some more tuning to fully address MC questions.