Skip to content

move get_transcript method to video service #37635

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
Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

move get_transcript method to video service #37635

wants to merge 2 commits into from
+75 −19

Conversation

@farhan
Copy link
Contributor

@farhan farhan commented Nov 14, 2025 •
edited
Loading

openedx/public-engineering#430

Testing instructions

Go to studio, and in a course, Stream any video having transcript. Transcripts should load properly.
Do the same with a v2 library Video Unit.

@kdmccormick kdmccormick force-pushed the farhan/move-transcripts-utils branch from 8b9f59c to 909c628 Compare November 14, 2025 18:04
Base automatically changed from farhan/move-transcripts-utils to master November 14, 2025 18:26
@farhan farhan force-pushed the farhan/move-get-transcript branch from a2e06e0 to 539d770 Compare November 17, 2025 13:23
@farhan farhan force-pushed the farhan/move-get-transcript branch 5 times, most recently from 657f69d to d77bac0 Compare November 25, 2025 15:34
@farhan farhan force-pushed the farhan/move-get-transcript branch from d77bac0 to 7e9576f Compare November 25, 2025 15:40
@farhan farhan added the create-sandbox open-craft-grove should create a sandbox environment from this PR label Nov 25, 2025
@open-craft-grove
Copy link

open-craft-grove commented Nov 25, 2025

Sandbox deployment failed 💥
Please check the settings and requirements.
Retry deployment by pushing a new commit or updating the requirements/settings in the pull request's description.
📜 Failure Logs
ℹ️ Grove Config, Tutor Config, Tutor Requirements

@open-craft-grove
Copy link

open-craft-grove commented Nov 26, 2025

Sandbox deployment failed 💥
Please check the settings and requirements.
Retry deployment by pushing a new commit or updating the requirements/settings in the pull request's description.
📜 Failure Logs
ℹ️ Grove Config, Tutor Config, Tutor Requirements

@open-craft-grove
Copy link

open-craft-grove commented Nov 26, 2025

Sandbox deployment failed 💥
Please check the settings and requirements.
Retry deployment by pushing a new commit or updating the requirements/settings in the pull request's description.
📜 Failure Logs
ℹ️ Grove Config, Tutor Config, Tutor Requirements

@farhan farhan closed this Nov 26, 2025
@farhan farhan reopened this Nov 26, 2025
@open-craft-grove
Copy link

open-craft-grove commented Nov 26, 2025

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

@farhan farhan marked this pull request as ready for review November 26, 2025 11:49
@farhan farhan requested review from feanil and kdmccormick November 26, 2025 11:49
@farhan
Copy link
Contributor Author

farhan commented Nov 26, 2025 •
edited
Loading

@kyle McCormick Following commit is the fix of a test case. I need your assistance in fixing it.
Context: Following is the fix of a djangoapps/content_libraries test case that needs VideoConfigService in the runtime.
7e9576f

@farhan farhan mentioned this pull request Nov 26, 2025
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@irtazaakram irtazaakram Awaiting requested review from irtazaakram irtazaakram is a code owner

@salman2013 salman2013 Awaiting requested review from salman2013 salman2013 is a code owner

@feanil feanil Awaiting requested review from feanil

@kdmccormick kdmccormick Awaiting requested review from kdmccormick

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.

3 participants

@farhan @open-craft-grove