Skip to content

Conversation

@connorhaugh
Copy link
Contributor

This fix: #680
created issues in the following xblocks:

The following xblocks on edX need to be updated:
Oppia-xblock (currently archived and read only, https://github.com/edx/oppia-xblock )
schoolyourself (https://github.com/openedx/schoolyourself-xblock )
imagemodal (https://github.com/openedx/xblock-image-modal )
invideoquiz (https://github.com/openedx/xblock-in-video-quiz )
concept (https://github.com/openedx/ConceptXBlock )
audio (https://github.com/openedx/AudioXBlock )
animation (currently archived and ready only, https://github.com/openedx-unsupported/AnimationXBlock )

Therefore, rather than change all of these, we needed to expeditiously find a solution, which is reverting the depr in this case. The added pain of the passthrough is outweighed by the lift to play whack-a-mole on every xblock.

@connorhaugh connorhaugh requested a review from feanil April 18, 2024 18:28
@connorhaugh connorhaugh merged commit 2ae4557 into master Apr 19, 2024
@connorhaugh connorhaugh deleted the feat--re-add-fragment-as-passthrough branch April 19, 2024 13:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants