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] process webcontent URLs in links and iframes [MER-2000] #154

Closed andersweinstein closed 1 year ago

andersweinstein commented 1 year ago

Adds webcontent URLS found in iframe src or link href to the set translated to media URLs, w/referenced files included in media manifest for uploading.

andersweinstein commented 1 year ago

On identification of webcontent links, I thought about this issue, but then just followed the model of nearby code for other elements which just tests for "includes('webcontent'). It looked to me like a decent heuristic that is extremely unlikely to ever fail in our use. I am not certain all webcontent links must begin with dots, though that is usual. There is a predicate in the code isLocalReference that could be used instead.