Skip to content

Lift and shift Video Block code moving from edx-platform #141

Merged
farhan merged 1 commit intofarhan/video-blockfrom
farhan/video-block-lift-and-sift
Feb 24, 2026
Merged

Lift and shift Video Block code moving from edx-platform #141
farhan merged 1 commit intofarhan/video-blockfrom
farhan/video-block-lift-and-sift

Conversation

@farhan
Copy link
Contributor

@farhan farhan commented Jan 23, 2026

Ticket

Followup/Successor PR

Details

This PR contains the code that has been moved (lifted and shifted) from the edx-platform to this repository.

  • Some of the files has been exactly copied as a whole.
  • Some of the parts of the code/methods has been copied; reason is to avoid noise and make a difference of the required code only which can be seen in the successor PR.

Source files references has also been added at the top of the files. If some more references are required, feel free to ask in this PR.

How to test:

  • We just have to check that the required code has been lifted and shifted correctly in this PR.
  • No need to verify logic in this PR, it can be verified in the next PR.

@farhan farhan force-pushed the farhan/video-block-lift-and-sift branch from 78fa647 to d684cf3 Compare January 23, 2026 12:21
@farhan farhan changed the title Lift and shift code of Video Block from edx-platform to xblock-contrib Lift and shift Video Block code moving from edx-platform Jan 26, 2026
@farhan farhan force-pushed the farhan/video-block-lift-and-sift branch from d684cf3 to 0c374ea Compare February 10, 2026 12:19
@farhan farhan force-pushed the farhan/video-block-lift-and-sift branch 2 times, most recently from de657bb to adda59b Compare February 20, 2026 05:55
@farhan farhan changed the base branch from main to farhan/video-block February 20, 2026 12:14
@farhan farhan force-pushed the farhan/video-block-lift-and-sift branch 2 times, most recently from 7e78463 to 103108d Compare February 24, 2026 09:10
This commit contains original code copied from edx-platform.
Purpose of keeping the first commit with original code is to get help in the review process.
Reviewer just needs to open the second commit and code changes will be clearly visible.
@farhan farhan force-pushed the farhan/video-block-lift-and-sift branch from 103108d to 39d7546 Compare February 24, 2026 09:13
@farhan farhan merged commit 01179ec into farhan/video-block Feb 24, 2026
3 of 11 checks passed
@farhan farhan deleted the farhan/video-block-lift-and-sift branch February 24, 2026 10:50
farhan added a commit that referenced this pull request Mar 2, 2026
This commit contains original code copied from edx-platform.
Purpose of keeping the first commit with original code is to get help in the review process.
Reviewer can easily view the next commit to view the changes done in xblocks-contrib while
shifting code from the edx-platform
farhan added a commit that referenced this pull request Mar 3, 2026
This commit contains original code copied from edx-platform.
Purpose of keeping the first commit with original code is to get help in the review process.
Reviewer can easily view the next commit to view the changes done in xblocks-contrib while
shifting code from the edx-platform
farhan added a commit that referenced this pull request Mar 4, 2026
* Lift and shift code of Video Block from edx-platform (#141)
This commit contains original code copied from edx-platform.
Purpose of keeping the first commit with original code is to get help in the review process.
Reviewer can easily view the next commit to view the changes done in xblocks-contrib while
shifting code from the edx-platform

* Update and fix the shifted Video XBlock code to make it run (#142)

* fix: Fix quality issues of the Video Block (#143)
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.

2 participants