search

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] Properly handle objectives and skill attachments #78

Closed darrensiegel closed 2 years ago

darrensiegel commented 2 years ago

This PR fixes a couple of problems related to ingestion of the learning model.

  1. LOs attached to pages were not showing up in Torus. Root cause was that the id of Objective resource was not being set to the "globalized" id. That was a single line fix (line 138 of convert.ts.
  2. After fixed 1 above, the objective hierarchy was not being preserved, and attached skills were not being preserved because those attributes still used the "local" id. Those now get translated to the global id.

Closes #66