Simon-Initiative / course-digest

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

[BUGFIX] handle pullout type [MER-2162] #187

Closed andersweinstein closed 1 year ago

andersweinstein commented 1 year ago

Pullouts can have a "type" field. Possible values include "tosumup" (To Sum Up). The pullout type attribute was not getting renamed, leading to undefined content element errors on ingestion.

In legacy, the type label is printed in bold followed by ellipses ("To Sum Up...") as a prefix to the pullout content, so this PR implements a version of that.

There are other divergences from legacy pullout behavior that are not addressed:

But this gets past the error-causing handling of the type field and gets the pullout content w/label on the page in readable form.