Skip to content

fix: media status page#1682

Open
Julusian wants to merge 1 commit intoSofie-Automation:mainfrom
SuperFlyTV:fix/media-status-page
Open

fix: media status page#1682
Julusian wants to merge 1 commit intoSofie-Automation:mainfrom
SuperFlyTV:fix/media-status-page

Conversation

@Julusian
Copy link
Member

@Julusian Julusian commented Mar 11, 2026

About the Contributor

This pull request is posted on behalf of Superfly

Type of Contribution

This is a: Bug fix

Current Behaviour

The media status page always shows No Media required by this system

New Behavior

image

I am not 100% sure whether it is fully fixed, as it sets up many subscriptions and tracing which subscriptions it expects to be active is non-trivial

Testing

  • I have added one or more unit tests for this PR
  • I have updated the relevant unit tests
  • No unit test changes are needed for this PR

Affected areas

Time Frame

Other Information

Status

  • PR is ready to be reviewed.
  • The functionality has been tested by the author.
  • Relevant unit tests has been added / updated.
  • Relevant documentation (code comments, system documentation) has been added / updated.

@Julusian Julusian added the Contribution from SuperFly.tv Contributions sponsored by SuperFly.tv label Mar 11, 2026
@codecov
Copy link

codecov bot commented Mar 11, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

@nytamin nytamin self-requested a review March 16, 2026 08:09
@nytamin
Copy link
Member

nytamin commented Mar 16, 2026

@Julusian could you update the description to add a "current behavior" / bug description?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Contribution from SuperFly.tv Contributions sponsored by SuperFly.tv

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants