Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: re-add fragment as passthrough to webfragement.fragment #739

Merged
merged 1 commit into from
Apr 19, 2024

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
9 of 10 checks passed
@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.

3 participants