[Test failure] TC_AUTHOR_45: SCORM not working

[angonz]

Release

Sumac

Expected behavior

SCORM should work after inserting a SCORM component.

Actual behavior

Studio shows this error:
Error: 'openedxscorm.scormxblock' is not a package

Steps to reproduce

In Studio, open or create a course.
Add "scorm" to the list of advanced modules.
In a unit, create a SCORM component from the advanced list.

Additional information

No response

[AntonellaPerrupato]

@mariajgrimaldi after adding the scorm in advanced settings, I was able to reproduce the error

[mariajgrimaldi]

@AntonellaPerrupato: can we try finding out a bit more about the issue? I think Scorm is not installed in the sandbox by default, but we'd have to confirm.

EDIT: it is installed by default: https://github.com/overhangio/tutor/blob/88652a4bb40519383bb6abe911472a546c3d9dc9/tutor/templates/build/openedx/Dockerfile#L107. We'll have to look into this a bit more.

[mariajgrimaldi]

@DawoudSheraz: can we confirm that https://github.com/overhangio/openedx-scorm-xblock was properly installed in the image used by the sandbox?

[DawoudSheraz]

@DawoudSheraz: can we confirm that https://github.com/overhangio/openedx-scorm-xblock was properly installed in the image used by the sandbox?

It is installed as part of image build, let me try re-building as we have recently pushed scorm v19 and it might not be available on sandbox yet.

[DawoudSheraz]

I installed scorm-xblock v19 on sandbox but the error is still there https://studio.sumac.demo.edly.io/container/block-v1:Test+tc001+2024+type@vertical+block@5350e8913b5f40f098c4b77571738cfd. I am informing the relevant folks to understand what might be going on here.
cc: @Danyal-Faheem

[mariajgrimaldi]

@DawoudSheraz: thank you so much!

[DawoudSheraz]

@mariajgrimaldi A new version of scorm-xblock has been released, this is fixed now https://studio.sumac.demo.edly.io/container/block-v1:Test+tc001+2024+type@vertical+block@5350e8913b5f40f098c4b77571738cfd#.

[mariajgrimaldi]

@angonz @AntonellaPerrupato: could you repeat the test case? Thank you!

@DawoudSheraz: thanks a lot for looking into this!

[AntonellaPerrupato]

@mariajgrimaldi this seems to be working now

[mariajgrimaldi]

I'll close this issue as resolved. Thank you all!