Skip to content

Stop relying on label to decide the file is VTT - #66

Open
JackLewis-digirati wants to merge 1 commit into
mainfrom
fix/tightenVttDetection
Open

JackLewis-digirati wants to merge 1 commit into
mainfrom
fix/tightenVttDetection

Conversation

@JackLewis-digirati

@JackLewis-digirati JackLewis-digirati commented Sep 16, 2026

Copy link
Copy Markdown
Contributor

What does this change?

Previously, label was used to figure out if the file was in the VTT format, along with checking for various strings in it. This caused issues as it would decide this label is VTT:

{
"label": {
      "en": [
          "METS-ALTO transcription"
      ]
  }
}

This tightens up that validation to only rely on profile and label, as well as remove some redundant checks on text/vtt, as it already checks these same fields for vtt

@JackLewis-digirati
JackLewis-digirati requested a review from a team as a code owner September 16, 2026 14:24
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