Conversation
farhan
added
the
create-sandbox
farhan
changed the title
farhan
changed the title
farhan
force-pushed
the
farhan/sandbox-3
branch
from
e9e1725 to
ea1ef4b
Compare
|
@farhan I completed the video xBlock testing. Video xBlock functionalities are working properly. I mentioned all the tests detailed in the Google doc. |
salman2013
left a comment
salman2013
left a comment
There was a problem hiding this comment.
The QA and dev side testing looks satisfactory to me.
kdmccormick
left a comment
kdmccormick
left a comment
There was a problem hiding this comment.
With the annotation fixed, LGTM 🚀
When you post in #risky-changes, please ping me and Peter Pinch (from MITx, they run close to master) and explain how the extracted block can be toggled off, in case they'd like to test first on their own, or in case of regression. Also add the same note in to Default Changes for Verawood please.
This is a long time coming. Great work!
| # .. toggle_creation_date: 2024-11-10 | ||
| # .. toggle_target_removal_date: 2025-06-01 | ||
| USE_EXTRACTED_VIDEO_BLOCK = False | ||
| USE_EXTRACTED_VIDEO_BLOCK = True |
There was a problem hiding this comment.
Can you update # .. toggle_default: False in the annotation above?
There was a problem hiding this comment.
let me do in another PR, it needs for other enabled blocks too
4 participants
Purpose of this PR is to create a sandbox on Extracted Video Block.
Once all testing is done, we can merge this PR to enable the extracted video block
Main ticket
Sandbox details for this PR:
sandbox-details
Manual Test cases document created by Aximprovements team:
word-document
Test data / Sample online video urls for the testing
Other details
PR for creating the sandbox for the Built-In Video Block:
Purpose of this sandbox is to compare the behaviour between extracted and built-in video block
#38158
Past testing:
Some testing has already been done in the past, here is the comment
Known Issue in the sandbox:
After experiencing issues (for example uploading transcript in the content-library video) in the sandbox (sandbox-1) of this PR I had created another sandbox in this PR (saying sandbox-2) for the testing.
I didn't experience any issue while doing my testing on:
Test Results:
Testing is done by @abdullah8590 and here are the results
I have done my testing on the following Video formats: