Skip to content

feat(types): add VideoSubtitleInfo types to playback info#1191

Draft
sjelfull wants to merge 1 commit intomainfrom
dam-video-subtitle-types
Draft

feat(types): add VideoSubtitleInfo types to playback info#1191
sjelfull wants to merge 1 commit intomainfrom
dam-video-subtitle-types

Conversation

@sjelfull
Copy link
Member

Summary

Adds subtitle type definitions for the video playback info response, split out from #1186 (renditions PR) to keep concerns separate.

Types added

  • VideoSubtitleInfoPublic — trackId, languageCode, url
  • VideoSubtitleInfoSigned — extends public with token + expiresAt
  • VideoSubtitleInfo — union type
  • subtitles?: VideoSubtitleInfo[] field on VideoPlaybackInfo

Re-exported from @sanity/client/media-library.

@vercel
Copy link

vercel bot commented Feb 26, 2026

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

1 Skipped Deployment
Project Deployment Actions Updated (UTC)
tsdocs-client Ignored Ignored Feb 26, 2026 8:56pm

Request Review

@github-actions
Copy link
Contributor

github-actions bot commented Feb 26, 2026

Coverage Report

Status Category Percentage Covered / Total
🔵 Lines 88.06% 3979 / 4518
🔵 Statements 88.06% 3979 / 4518
🔵 Functions 85.67% 317 / 370
🔵 Branches 89.13% 1173 / 1316
File Coverage
File Stmts Branches Functions Lines Uncovered Lines
Changed Files
src/media-library.ts 90.47% 75% 100% 90.47% 58-59
src/types.ts 100% 100% 100% 100%
Generated in workflow #4079 for commit ff01a0c by the Vitest Coverage Report Action

- Add VideoSubtitleInfoPublic, VideoSubtitleInfoSigned, and
  VideoSubtitleInfo union type for subtitle tracks
- Add subtitles field to VideoPlaybackInfo interface
- Re-export subtitle types from media-library entry point

Co-Authored-By: Claude <noreply@anthropic.com>
@sjelfull sjelfull force-pushed the dam-video-subtitle-types branch from 327989f to ff01a0c Compare February 26, 2026 20:56
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