Skip to content

Match more videos - #4783

Merged
marcoacierno merged 1 commit into
mainfrom
trim-file-name
Aug 14, 2026
Merged

marcoacierno merged 1 commit into
mainfrom
trim-file-name

Conversation

@marcoacierno

Copy link
Copy Markdown
Member

What

ToDo

@vercel

vercel Bot commented Aug 14, 2026 •

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
pycon Ready Ready Preview Aug 14, 2026 10:25pm

@marcoacierno
marcoacierno merged commit 853e584 into main Aug 14, 2026
8 of 9 checks passed
@marcoacierno
marcoacierno deleted the trim-file-name branch August 14, 2026 22:25
@claude

claude Bot commented Aug 14, 2026

Copy link
Copy Markdown
Contributor

Adds day-aware disambiguation and diacritic-insensitive matching to the video-upload path matcher, plus a fallback for multi-speaker talks uploaded under a single speaker's name.

Day-number computation silently excludes custom-type items (backend/conferences/admin/conference.py, the talk_days query only includes submission, talk, keynote, panel). If a conference day has only custom items (e.g. a day that's just lightning talks, or an opening/closing-only day) and no talk/keynote/panel/submission item, that day won't be counted as a "talk day," which shifts the day numbers assigned to all subsequent days relative to how the video files are actually named (D1/D2/...). The new test happens to avoid this because every day with lightning talks (custom) also has a talk-type item scheduled the same day, so the gap isn't exercised. Worth confirming this matches how schedules are actually built, or including custom in the day-counting filter.

@codecov

codecov Bot commented Aug 14, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 84.61538% with 4 lines in your changes missing coverage. Please review.
✅ Project coverage is 92.88%. Comparing base (f4cec7f) to head (ec7dbdd).
⚠️ Report is 1 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #4783      +/-   ##
==========================================
+ Coverage   92.86%   92.88%   +0.02%     
==========================================
  Files         360      360              
  Lines       11171    11192      +21     
  Branches      894      901       +7     
==========================================
+ Hits        10374    10396      +22     
  Misses        675      675              
+ Partials      122      121       -1     
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

This branch was successfully deployed

1 active deployment
Preview — ec7dbdd6 Deployed Aug 14, 2026 by vercel[bot]
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.

1 participant