Skip to content

Media source: remove mention of supervised, as it is deprecated#41946

Merged
frenck merged 1 commit into
currentfrom
media-source-remove-supervised
Nov 24, 2025
Merged

Media source: remove mention of supervised, as it is deprecated#41946
frenck merged 1 commit into
currentfrom
media-source-remove-supervised

Conversation

@c0ffeeca7
Copy link
Copy Markdown
Contributor

Media source: remove mention of supervised

  • as this installation method has been deprecated

Proposed change

Type of change

  • Spelling, grammar or other readability improvements (current branch).
  • Adjusted missing or incorrect information in the current documentation (current branch).
  • Added documentation for a new integration I'm adding to Home Assistant (next branch).
  • Added documentation for a new feature I'm adding to Home Assistant (next branch).
  • Removed stale or deprecated documentation.

Additional information

  • Link to parent pull request in the codebase:
  • Link to parent pull request in the Brands repository:
  • This PR fixes or closes issue: fixes #

Checklist

  • This PR uses the correct branch, based on one of the following:
    • I made a change to the existing documentation and used the current branch.
    • I made a change that is related to an upcoming version of Home Assistant and used the next branch.
  • The documentation follows the Home Assistant documentation standards.

Copilot AI review requested due to automatic review settings November 24, 2025 14:34
@c0ffeeca7 c0ffeeca7 requested a review from hunterjm as a code owner November 24, 2025 14:34
@home-assistant home-assistant Bot added the current This PR goes into the current branch label Nov 24, 2025
@netlify
Copy link
Copy Markdown

netlify Bot commented Nov 24, 2025

Deploy Preview for home-assistant-docs ready!

Name Link
🔨 Latest commit e629c54
🔍 Latest deploy log https://app.netlify.com/projects/home-assistant-docs/deploys/69246d12b0a95f000893b6af
😎 Deploy Preview https://deploy-preview-41946--home-assistant-docs.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR updates the media source integration documentation to remove references to the deprecated "Supervised" installation method, following the official deprecation announced in May 2025. This ensures the documentation only references the two remaining supported installation methods: Home Assistant OS and Home Assistant Container.

Key changes:

  • Removed mentions of "Supervised" from the local media folder access instructions
  • Simplified the explanation to focus only on OS and Container installation methods

Comment on lines +37 to 38
This folder is by default configured in the path `/media`.

Copy link

Copilot AI Nov 24, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The sentence "This folder is by default configured in the path /media." is now redundant with the previous paragraph which already states "By default, the integration looks for media in a specified folder" followed by information about the /media/local path.

Consider removing this line entirely or merging it more naturally with the surrounding context. For example:

By default, the integration looks for media in the `/media` folder. If other `media_dirs` are not declared you need to use `/media/local` path for example in companion app notification.

Home Assistant OS users can access this folder by using, for example, the Samba add-on. Users of Home Assistant Container can mount a volume of their choice to `/media`.
Suggested change
This folder is by default configured in the path `/media`.

Copilot uses AI. Check for mistakes.
Comment on lines 40 to 41
for example, the Samba add-on. Users of Home Assistant Container can
mount a volume of their choice to `/media`.
Copy link

Copilot AI Nov 24, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The phrasing "Users of Home Assistant Container" is inconsistent with "Home Assistant OS users" in the previous sentence. For better readability and consistency, consider using parallel structure:

Home Assistant OS users can access this folder by using, for example, the Samba add-on. Home Assistant Container users can mount a volume of their choice to `/media`.

This maintains consistent phrasing style between the two installation methods.

Copilot generated this review using guidance from repository custom instructions.
Copy link
Copy Markdown
Member

@frenck frenck left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks, @c0ffeeca7 👍

../Frenck

                       

Blogging my personal ramblings at frenck.dev

@frenck frenck merged commit adbdf49 into current Nov 24, 2025
15 checks passed
@frenck frenck deleted the media-source-remove-supervised branch November 24, 2025 16:18
@github-actions github-actions Bot locked and limited conversation to collaborators Nov 25, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

current This PR goes into the current branch

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants