Converts LTI views to React

ucfopen / Materia

Engage students with easily embedded apps for online courses. Supercharge your course with compelling experiences and game mechanics.

https://ucfopen.github.io/Materia-Docs/

GNU Affero General Public License v3.0

36 stars 33 forks source link

Converts LTI views to React #1415

Closed cayb0rg closed 1 year ago

cayb0rg commented 1 year ago

Changes

Converts LTI views to React
Removes jquery from test_provider and test_learner pages
Updates deprecated assertContains to assertContainsEquals in test functions

clpetersonucf commented 1 year ago

Updated with changes to the main react branch. Something I'm noticing: the LTI picker isn't loading any widgets, which I suspect is due to pagination in the widget_paginate_instances_get, but haven't looked at it in detail.

clpetersonucf commented 1 year ago

The only other thing I'm noticing is that attempting to test LTI Assignment Launch in the test provider yields the message LTI Assignment URL can not be blank! even when the the assignment URL is populated.

Update: this is probably because the test provider was relying on jquery for basic functionality, which is no longer bundled with Materia. We can probably update the page to use a CDN link for jQuery. I'd rather not bundle it with Materia's JS and CSS assets.