feat: add patch endpoint to update transcript providers #562
+191
−1
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## master #562 +/- ##
==========================================
+ Coverage 94.58% 94.65% +0.06%
==========================================
Files 32 32
Lines 3178 3256 +78
Branches 122 126 +4
==========================================
+ Hits 3006 3082 +76
- Misses 154 156 +2
Partials 18 18
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
hajorg
reviewed
Feb 12, 2025
hajorg
reviewed
Feb 12, 2025
hajorg
approved these changes
Feb 12, 2025
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
What this does:
This PR adds a DRF Patch endpoint for updating the "provider" field on VideoTranscript records.
Context:
When 2u/Edx wanted to add ai-translated content to the platform part of the project was translating the video transcripts. During the creation of the translated transcripts they were labeled with the
provider="Custom"which has turned out to be too broad for some of the use cases Edx is looking to build out. We have since added a new provider in this repo and are looking to apply it with this change.The status of the translated transcripts live in an external service (
IDAin edx terms) and needs access toedx-valvia the REST interface which is the core reason behind this change.This is in regards to the following 2U ticket.
https://2u-internal.atlassian.net/browse/AU-2381