Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update modbus.markdown #24594

Closed
wants to merge 118 commits into from
Closed

Update modbus.markdown #24594

wants to merge 118 commits into from

Conversation

Nippey
Copy link
Contributor

@Nippey Nippey commented Oct 17, 2022

Update binary sensor to accept holding registers as input type

Proposed change

I have a 32-channel input ("N4DIM32") for modbus. This device exposes its internal states via holding registers.
(If required, I can attach the interface description provided by the seller.)
As binary_sensor currently only supports CALL_TYPE_COIL, CALL_TYPE_DISCRETE:
ADDED: Support for CALL_TYPE_REGISTER_HOLDING.

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

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.

bdraco and others added 30 commits September 7, 2022 12:53
Co-authored-by: Franck Nijhof <frenck@frenck.nl>
* SkyConnect landing page

* Inches

* Final tweaks

* Update source/skyconnect/index.html

Co-authored-by: Joakim Sørensen <joasoe@gmail.com>

Co-authored-by: Zack Barett <zackbarett@hey.com>
Co-authored-by: Joakim Sørensen <joasoe@gmail.com>
@home-assistant home-assistant bot removed current This PR goes into the current branch needs-rebase The PR has been branched of the wrong base branch or targets an incorrect target branch labels Oct 17, 2022
@home-assistant home-assistant bot added the next This PR goes into the next branch label Oct 17, 2022
@Nippey
Copy link
Contributor Author

Nippey commented Oct 17, 2022

I think I messed up. I will redo on "next".

@Nippey Nippey closed this Oct 17, 2022
@home-assistant home-assistant bot removed in-progress This PR/Issue is currently being worked on Hacktoberfest An PR on this issue (or the PR itself) is eligible towards Hacktoberfest! labels Oct 17, 2022
@github-actions github-actions bot locked and limited conversation to collaborators Oct 18, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
has-parent This PR has a parent PR in a other repo next This PR goes into the next branch
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

None yet