Skip to content

fix: allow admin broadcasts and warn about missing host logins - #311

Merged
anneoneone merged 4 commits into
mainfrom
fix/301-admin-show-broadcast
Sep 27, 2026
Merged

anneoneone merged 4 commits into
mainfrom
fix/301-admin-show-broadcast

Conversation

@anneoneone

Copy link
Copy Markdown
Collaborator
  • Let admins broadcast any show while keeping hosts limited to their assignments.
  • Warn before a show starts when its host has no recorded login.
  • Fix small-image brightness analysis and two TypeScript errors.

Tests: 168 frontend and 89 backend tests pass; 2 credential-dependent tests remain skipped. Frontend lint, type-checking, build, and Rust formatting pass.

Closes: #301

Implementation and validation details
  • Check show access before WebSocket upgrades and restrict stream stop and takeover.
  • Persist first login time and backfill from existing sessions. Deleted sessions cannot be recovered, so the warning says “no login recorded”.
  • Refresh host metadata without replacing unsaved edits. Pause refreshes on cached-page deactivation and reject stale responses.
  • Require the correct selected show before entering the live flow.
  • Sample at least one bottom row for brightness analysis. Narrow host-choice types and let FLV destroy handle detachment.
  • Regression tests cover role access, login persistence, actual WebSocket upgrades, stream stop, show selection, warning timing, reassignment, and cached-page lifecycle.
  • Existing strict Clippy diagnostics remain outside this change. GitNexus impact review completed; its later index refresh was blocked by another process holding the database lock.

@vercel

vercel Bot commented Sep 27, 2026

Copy link
Copy Markdown

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

Project Deployment Actions Updated
live-moafunk-de Ready Ready Preview Sep 27, 2026 12:23am UTC

This branch was successfully deployed

1 active deployment
Preview — 1ca191cc Deployed Sep 27, 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.

Let admins go live for any show; warn when the host has never logged in

1 participant