Skip to content

enable extracted video block #37851

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.

Sign up for GitHub

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

Jump to bottom
Draft
farhan wants to merge 6 commits into
base: master
Choose a base branch
from
Draft

enable extracted video block #37851

farhan wants to merge 6 commits into from
+72 −26

Conversation

@farhan
Copy link
Contributor

@farhan farhan commented Jan 8, 2026 •
edited
Loading

Relevant pr in xblocks-contrib: openedx/xblocks-contrib#83

Currently PR will stay in draft until all the test cases get passed and branch updates to pypi version of xblocks-contrib

@farhan farhan requested a review from ttqureshi January 8, 2026 07:24
@farhan farhan closed this Jan 8, 2026
@farhan farhan reopened this Jan 8, 2026
@farhan farhan closed this Jan 8, 2026
@farhan farhan reopened this Jan 8, 2026
@farhan farhan added the create-sandbox open-craft-grove should create a sandbox environment from this PR label Jan 9, 2026
@farhan farhan mentioned this pull request Jan 9, 2026
Closed
@open-craft-grove
Copy link

open-craft-grove commented Jan 9, 2026

Sandbox deployment successful 🚀
🎓 LMS
📝 Studio
ℹ️ Grove Config, Tutor Config, Tutor Requirements

@farhan farhan closed this Jan 9, 2026
@farhan farhan reopened this Jan 9, 2026
@farhan farhan closed this Jan 9, 2026
@farhan farhan reopened this Jan 9, 2026
@open-craft-grove
Copy link

open-craft-grove commented Jan 9, 2026

Sandbox deployment successful 🚀
🎓 LMS
📝 Studio
ℹ️ Grove Config, Tutor Config, Tutor Requirements

@open-craft-grove
Copy link

open-craft-grove commented Jan 9, 2026

Sandbox deployment successful 🚀
🎓 LMS
📝 Studio
ℹ️ Grove Config, Tutor Config, Tutor Requirements

@farhan farhan force-pushed the farhan/enable-video-block-4 branch from 3485191 to fad0bc5 Compare January 9, 2026 16:16
@farhan farhan mentioned this pull request Jan 12, 2026
Open
@open-craft-grove
Copy link

open-craft-grove commented Jan 12, 2026

Sandbox deployment successful 🚀
🎓 LMS
📝 Studio
ℹ️ Grove Config, Tutor Config, Tutor Requirements

@farhan farhan closed this Jan 12, 2026
@farhan farhan reopened this Jan 12, 2026
@open-craft-grove
Copy link

open-craft-grove commented Jan 13, 2026

Sandbox deployment successful 🚀
🎓 LMS
📝 Studio
ℹ️ Grove Config, Tutor Config, Tutor Requirements

@farhan farhan mentioned this pull request Jan 13, 2026
Closed
@farhan farhan closed this Jan 13, 2026
@farhan farhan reopened this Jan 13, 2026
@farhan farhan mentioned this pull request Jan 14, 2026
Closed
@farhan farhan closed this Jan 15, 2026
@farhan farhan reopened this Jan 15, 2026
@farhan farhan closed this Jan 15, 2026
@farhan farhan reopened this Jan 15, 2026
@farhan farhan force-pushed the farhan/enable-video-block-4 branch from f35948f to 477a183 Compare January 16, 2026 08:45
@ttqureshi
Copy link
Member

ttqureshi commented Jan 16, 2026 •
edited
Loading

Video XBlock import/export testing notes

I have tested the import/export functionality of the Extracted Video XBlock on my local setup. Based on my testing, the Extracted Video XBlock is exported in the pointer tag format during export, and when imported, it parses the pointer tag format correctly. The functionality is working as expected.
I have tested the following formats of the Video XBlock:

  1. Youtube Video
  2. Youtube Video with Transcript
  3. mp4 Video
  4. webm Video

The exported course with the built-in Video XBlock is the same as the course with the extracted Video XBlock. This shows that the extracted Video XBlock is being exported as expected.

cc: @farhan

@open-craft-grove
Copy link

open-craft-grove commented Jan 20, 2026

Sandbox deployment successful 🚀
🎓 LMS
📝 Studio
ℹ️ Grove Config, Tutor Config, Tutor Requirements

@farhan farhan mentioned this pull request Jan 23, 2026
Open
2 tasks
@farhan farhan force-pushed the farhan/enable-video-block-4 branch from 7e1a394 to 94d2642 Compare January 23, 2026 15:39
@open-craft-grove
Copy link

open-craft-grove commented Jan 23, 2026

Sandbox deployment successful 🚀
🎓 LMS
📝 Studio
ℹ️ Grove Config, Tutor Config, Tutor Requirements

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@ttqureshi ttqureshi Awaiting requested review from ttqureshi

@irtazaakram irtazaakram Awaiting requested review from irtazaakram irtazaakram will be requested when the pull request is marked ready for review irtazaakram is a code owner

@salman2013 salman2013 Awaiting requested review from salman2013 salman2013 will be requested when the pull request is marked ready for review salman2013 is a code owner

At least 1 approving review is required to merge this pull request.

Assignees

No one assigned

Labels

create-sandbox open-craft-grove should create a sandbox environment from this PR

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@farhan @open-craft-grove @ttqureshi