Skip to content
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

Open
wants to merge 6 commits into
base: dev/33-serpierite
Choose a base branch
from

Conversation

FrenjaminBanklin
Copy link
Contributor

@FrenjaminBanklin FrenjaminBanklin commented Sep 28, 2021

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.

…s and display them in the viewer. Added Materia as a type option for questions. Added support to assessment reviews for partial scores.
@FrenjaminBanklin FrenjaminBanklin changed the base branch from master to dev/26-neon-apatite September 28, 2021 13:53
Copy link
Member

@zachberry zachberry left a 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.

…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.
@FrenjaminBanklin FrenjaminBanklin changed the base branch from dev/26-neon-apatite to dev/28-jadeite March 3, 2022 13:48
@FrenjaminBanklin FrenjaminBanklin changed the base branch from dev/28-jadeite to dev/29-sodalite August 26, 2022 20:20
@stale
Copy link

stale bot commented Jun 18, 2023

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.

@stale stale bot added the stale label Jun 18, 2023
…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.
@stale stale bot removed the stale label Sep 27, 2023
@FrenjaminBanklin FrenjaminBanklin marked this pull request as ready for review September 27, 2023 15:35
@FrenjaminBanklin FrenjaminBanklin changed the base branch from dev/29-sodalite to dev/33-serpierite September 27, 2023 15:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Materia Chunk - Notify viewer client when a score has been received
2 participants