Skip to content

Added multiple video handling to frontend#2062

Open
delexagon wants to merge 3 commits intocodeforboston:mainfrom
delexagon:multi-video-frontend
Open

Added multiple video handling to frontend#2062
delexagon wants to merge 3 commits intocodeforboston:mainfrom
delexagon:multi-video-frontend

Conversation

@delexagon
Copy link
Collaborator

@delexagon delexagon commented Mar 12, 2026

Summary

Addresses the frontend work for #2015. This is meant to be cross-compatible with the existing design and the future design.

Checklist

  • On the frontend, I've made my strings translate-able.
  • I've made pages responsive and look good on mobile.

Screenshots

Captura de pantalla_20260312_082135

Known issues

I'm pretty sure I'm going to have to create a way to address longer (overflowing) names, and I haven't implemented that yet. I've tested this, but I'm probably going to want to look it over again once I get to the other side.

Steps to test/reproduce

  1. Check all existing transcription related features. There should be no visual or functional changes for hearings with only one video (other than the change to 'Loading transcript for this hearing...'.
  2. The way I've been testing multiple videos is by manually injecting JSON into hearing.ts. It's kind of awkward, but it works. If you want to do this, the JSON I've been using for testing is
videos: [
      {
        "title": "part 1",
        "url": "https://prodarchivevideo.blob.core.windows.net/video/2025/Hearings/Joint/July/01_2.mp4",
        "transcriptionId": "68feedc9-35ca-49ea-bf99-0fdb02e2f5a2",
        "fetchedAt":  {
            "seconds": 1752077206,
            "nanoseconds": 92000000
        }
      },
      {
        "title": "part 3",
        "url": "https://prodarchivevideo.blob.core.windows.net/video/2025/Hearings/Joint/July/01_2.mp4",
        "fetchedAt":  {
          "seconds": 1752077210,
          "nanoseconds": 92000000
        }
      },
      {
        "title": "part 2",
        "url": "https://prodarchivevideo.blob.core.windows.net/video/2025/Hearings/Joint/July/15_3.mp4",
        "transcriptionId": "d656eb4a-17c3-45c1-ab65-c9d75691b644",
        "fetchedAt":  {
          "seconds": 1752077210,
          "nanoseconds": 92000000
        }
      }
    ]
  1. With multiple videos, test that the transcription changes while moving between videos, both as download and as element. The transcript highlight should be reset when changing videos. The transcript highlight should always reflect the currently active video. The ?t= parameter should affect the video in ?v=, and should only be erased when changing the active video. The ?v= parameter should always reflect the currently active video. If a video has not had a transcription generated for it yet, it should correctly display that there is no transcript available. Moving to and from this state to a state with a transcript should not generate any errors. Etc.

@vercel
Copy link

vercel bot commented Mar 12, 2026

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

Project Deployment Actions Updated (UTC)
maple-dev Ready Ready Preview, Comment Mar 13, 2026 1:32am

Request Review

@delexagon
Copy link
Collaborator Author

FAIL src/malegislature.test.ts
Not sure why this happened

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