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

Bibliography migration failing #58

Closed gjbunyea closed 2 years ago

gjbunyea commented 2 years ago

When trying to migrate 2 courses (Wellstart v1.2, C@CM v3.4) I get a failure related to bibliography support on each

C:\Users\gregb\Documents\OLI\SVN\course-digest\src\resources\workbook.ts:79
      bibrefs.push(bibEntries.get(entry).id);
                                        ^
TypeError: Cannot read properties of undefined (reading 'id')
    at Element.<anonymous> (C:\Users\gregb\Documents\OLI\SVN\course-digest\src\resources\workbook.ts:79:41)
    at LoadedCheerio.each (C:\Users\gregb\Documents\OLI\SVN\course-digest\node_modules\cheerio\lib\api\traversing.js:480:26)
    at WorkbookPage.translate (C:\Users\gregb\Documents\OLI\SVN\course-digest\src\resources\workbook.ts:77:15)
    at C:\Users\gregb\Documents\OLI\SVN\course-digest\src\convert.ts:57:17
    at processTicksAndRejections (node:internal/process/task_queues:96:5)

These are the courses tested thus far https://svn.oli.cmu.edu/svn/content/editor/projects/echo-oli-cmu-edu/wellstart-v8fhrc86/branches/v_1_2-echo https://svn.oli.cmu.edu/svn/content/c_at_cm/c_at_cm/branches/v_3_4-echo

rgachuhi commented 2 years ago

This is now fixed and landed in master