Simon-Initiative / course-digest

Tool to produce a summary or digest of OLI course package contents
MIT License
2 stars 0 forks source link

[BUG FIX] chemistry focused content fixes, warnings #79

Closed darrensiegel closed 2 years ago

darrensiegel commented 2 years ago

This PR accounts for a variety of different "ingest failures":

  1. Media assets containing spaces within their filenames were breaking the digest tool. This is fixed, and the correct encoded form of the URL (with %20 encoded as the space) is stored for the media asset item
  2. Some activities contained a response that had no feedback. Adjusted the feedback creation to account for this and to correct and dummy, empty feedback in this case.
  3. Converted composite_activity title elements to paragraphs
  4. Previously, learning objectives and skills model files were to assumed to only exist in top level x-oli-learning_objectives and x-oli-skills_model directories. These directories, in reality, can exist anywhere in the content directory structure.
  5. Detect and adjust content in situations where the legacy content model was abused. For example, tables within paragraphs, lists within paragraphs, paragraphs within paragraphs, etc.
  6. Adds a mechanism for flagging conditions in resources as a collection of warnings
  7. Identifies and fixes MCQs with no correct answer
  8. Migrates <inquiry> elements to plain paragraphs.
  9. Migrates <term> inline as a term, will be specifically handled in Torus in upcoming PR.
  10. Adds support for migrating branching assessments.