Skip to content

Remove invalid instructions from jvcprojector#42568

Merged
c0ffeeca7 merged 1 commit intohome-assistant:currentfrom
SteveEasley:jvcprojector/remote_invalid_instructions
Dec 15, 2025
Merged

Remove invalid instructions from jvcprojector#42568
c0ffeeca7 merged 1 commit intohome-assistant:currentfrom
SteveEasley:jvcprojector/remote_invalid_instructions

Conversation

@SteveEasley
Copy link
Copy Markdown
Contributor

Proposed change

Removing some instructions that were not valid. The issue referenced was actually fixed the year before in home-assistant/core#126453. The user that reported the problem a year later was just sharing a similiar issue from another home automation app.

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 December 14, 2025 18:38
@home-assistant home-assistant Bot added current This PR goes into the current branch has-parent This PR has a parent PR in another repo in-progress This PR/Issue is currently being worked on needs-rebase The PR has been branched of the wrong base branch or targets an incorrect target branch labels Dec 14, 2025
@home-assistant
Copy link
Copy Markdown

It seems that this PR is targeted against an incorrect branch since it has a parent PR on one of our codebases. Documentation that needs to be updated for an upcoming release should target the next branch. Please change the target branch of this PR to next and rebase if needed.

@netlify
Copy link
Copy Markdown

netlify Bot commented Dec 14, 2025

Deploy Preview for home-assistant-docs ready!

Name Link
🔨 Latest commit 4c2a286
🔍 Latest deploy log https://app.netlify.com/projects/home-assistant-docs/deploys/693f0433bf08970008d25156
😎 Deploy Preview https://deploy-preview-42568--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 removes outdated and invalid password hashing instructions from the JVC Projector integration documentation. The instructions were for NZ800 and NZ900 models but are no longer needed since the underlying issue was fixed in a previous core PR (#126453). The removal improves documentation accuracy by eliminating instructions that would mislead users.

Key changes:

  • Removed the entire "Password hashing for newer models (NZ800 and NZ900)" subsection containing outdated workaround instructions
Comments suppressed due to low confidence (1)

source/_integrations/jvc_projector.markdown:44

  • The removed instructions previously directed users to paste their projector password (concatenated with JVCKWPJ) into a third-party online SHA256 hash generator, which leaks the password material to an external service. An attacker controlling or eavesdropping on that service could capture these secrets and reuse them to control the projector or any other system where the same password is used. Removing this guidance avoids encouraging users to transmit passwords to untrusted websites; if hashing instructions are needed in the future, they should rely on local tools or built-in functionality that never sends secrets off-device.
{% include integrations/config_flow.md %}

## Remote

The JVC Projector remote platform will create a [Remote](/integrations/remote/) entity for the device. This entity allows you to send the following commands via the [remote.send_command](/integrations/remote/) action.

- `menu`
- `up`
- `down`

Copy link
Copy Markdown
Contributor

@c0ffeeca7 c0ffeeca7 left a comment

Choose a reason for hiding this comment

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

Thank you, @SteveEasley 👍

@c0ffeeca7 c0ffeeca7 merged commit 32fe691 into home-assistant:current Dec 15, 2025
13 checks passed
@home-assistant home-assistant Bot removed in-progress This PR/Issue is currently being worked on needs-rebase The PR has been branched of the wrong base branch or targets an incorrect target branch labels Dec 15, 2025
konsulten pushed a commit to konsulten/home-assistant.io that referenced this pull request Dec 15, 2025
@github-actions github-actions Bot locked and limited conversation to collaborators Dec 16, 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 has-parent This PR has a parent PR in another repo

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants