Closed lduarte1991 closed 8 years ago
@lduarte1991 Very cool! Regarding point 9 - no need to put the preview user stuff in an ORG, since it shouldn't impact us at all. Canvas almost always provides either lis_person_name_full
or lis_person_sourcedid
so I imagine that case will only ever come up when you're running it in edX studio.
:+1:
@arthurian Have you gotten a chance to check this out? Not sure if the thumbs up was an okay to merge or just for the code haha
@lduarte1991 Good to merge!
This PR would bring the tool up to v1.2.0 with the addition of video annotations.
It contains quite a few changes to the video annotation tool as it existed in edX. Still following up on the design of the text and image tools, this adds one new element to the dashboard and contains the same screen-reader friendly navigation and structure.
Updates: