Skip to content

feat(broadcast): guard against accidentally killing the live display - #180

Open
Josesx506 wants to merge 1 commit into
openbezal:mainfrom
Josesx506:feat/live-output-guards
Open

feat(broadcast): guard against accidentally killing the live display#180
Josesx506 wants to merge 1 commit into
openbezal:mainfrom
Josesx506:feat/live-output-guards

Conversation

@Josesx506

Copy link
Copy Markdown

Two operator-safety guards around the Live toggle:

  • Disabling an output while Live is on now asks for confirmation (toast with a 'Close anyway' action) instead of cutting what the congregation sees on a single click.
  • Flipping Go Live with no broadcast window open warns the operator that nothing will be visible until a window is opened — an easy state to miss on the controller screen.

Claude-Session: https://claude.ai/code/session_01PAHGVvCBEcMPVtazz3cyoz

Description

Type of change

  • Bug fix
  • New feature
  • Refactoring (no functional changes)
  • Documentation
  • Build / CI
  • Performance improvement

Areas affected

  • Frontend (React / TypeScript)
  • Backend (Rust / Tauri commands)
  • Rust crate:
  • Remote control (OSC / HTTP)
  • Broadcast / NDI
  • Theme Designer
  • Bible data / search
  • Audio / STT

Checklist

  • I have tested this change locally
  • bun run typecheck passes
  • cargo clippy passes without warnings
  • bun run test passes (if applicable)
  • I have added tests for new functionality (if applicable)
  • UI changes include a screenshot or recording below

Tested on

  • macOS
  • Windows
  • Linux

Screenshots / recordings

Two operator-safety guards around the Live toggle:

- Disabling an output while Live is on now asks for confirmation (toast
  with a 'Close anyway' action) instead of cutting what the congregation
  sees on a single click.
- Flipping Go Live with no broadcast window open warns the operator that
  nothing will be visible until a window is opened — an easy state to
  miss on the controller screen.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01PAHGVvCBEcMPVtazz3cyoz
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