Skip to content

tools: added additional details on video upload#75

Open
destevae wants to merge 2 commits intomasterfrom
70-set-additional-details-on-video-upload
Open

tools: added additional details on video upload#75
destevae wants to merge 2 commits intomasterfrom
70-set-additional-details-on-video-upload

Conversation

@destevae
Copy link
Collaborator

#70: Added default language, audio language, license, and recording date details to Youtube API JSON request when uploading videos. Was not able to add the recording location since it was depreciated a while ago. In order to get recording date of the talk, I added date to the application's JSON from the database.

@destevae destevae requested a review from EaterOA March 17, 2025 23:36
@destevae destevae linked an issue Mar 17, 2025 that may be closed by this pull request
tools/upload.py Outdated
date = room_day["date"]

room_day_name = f"{rdash(room)}-{rdash(day)}"
room_date = datetime.strptime(date, '%a, %d %b %Y %H:%M:%S %Z').isoformat()
Copy link
Collaborator

Choose a reason for hiding this comment

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

You should convert the Date to an isoformat in the generate json endpoints, instead of letting (I think) Flask default serialize it as this format that you have to parse and reformat.

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.

set additional details on video upload

3 participants