openequella / openEQUELLA-blackboard-integration

0 stars 3 forks source link

Migration-b2: Parser not recognizing older integration data #32

Closed cbeach47 closed 4 years ago

cbeach47 commented 4 years ago

An adopter has found that older integration data produces LTI links of the form https://my-oeq-url/null. Also, the current behavior has shown room for improvement.

Looking to make the following changes:

1 Turn off the deletion of data from the old integration building block. It should not affect the migrated link since it's now under a new handle (eg apereo/oeq), and deleting it may create unforeseen issues.

2 Implement the following logic flow:

3 Log that the script is ignoring the content id / handle if it DOESN'T match:

4 Review the migration of the link description.

cbeach47 commented 4 years ago

Phase 1 completed and included in the release https://github.com/openequella/openEQUELLA-blackboard-integration/releases/tag/v2.1.1

cbeach47 commented 4 years ago

Phase 2 completed in v2.1.2 -

Enhancements: