Skip to content

Updates the backpopulate_video_shorts command to be compatible with the latest serializer changes#2955

Merged
mbertrand merged 1 commit into
mainfrom
mb/fix_backpopulate_shorts
Feb 17, 2026
Merged

Updates the backpopulate_video_shorts command to be compatible with the latest serializer changes#2955
mbertrand merged 1 commit into
mainfrom
mb/fix_backpopulate_shorts

Conversation

@mbertrand
Copy link
Copy Markdown
Member

@mbertrand mbertrand commented Feb 13, 2026

What are the relevant tickets?

Related to https://github.com/mitodl/hq/issues/8880

Description (What does it do?)

Fixes the backpopulate_video_shorts command which was based on an outdated serializer definition. It's only intended for local development at this point though.

How can this be tested?

-Set the following in your backend.local.env file:

AWS_S3_CUSTOM_DOMAIN="rc.learn.mit.edu"
AWS_S3_PREFIX="media"

Comment thread video_shorts/api.py
@mbertrand mbertrand force-pushed the mb/fix_backpopulate_shorts branch 2 times, most recently from cb3b6e2 to f9c2b1d Compare February 13, 2026 16:09
Comment thread video_shorts/api.py
@mbertrand mbertrand added the Needs Review An open Pull Request that is ready for review label Feb 13, 2026
@mbertrand mbertrand force-pushed the mb/fix_backpopulate_shorts branch from f9c2b1d to 0202101 Compare February 13, 2026 16:51
@mbertrand mbertrand force-pushed the mb/fix_backpopulate_shorts branch from 0202101 to dbaedf1 Compare February 13, 2026 16:52
@shanbady shanbady self-requested a review February 17, 2026 14:45
Copy link
Copy Markdown
Contributor

@shanbady shanbady left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM 👍

@shanbady shanbady added Waiting on author and removed Needs Review An open Pull Request that is ready for review labels Feb 17, 2026
@mbertrand mbertrand merged commit 1cef467 into main Feb 17, 2026
14 checks passed
@mbertrand mbertrand deleted the mb/fix_backpopulate_shorts branch February 17, 2026 15:54
@odlbot odlbot mentioned this pull request Feb 17, 2026
7 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants