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

Smart plugs: add ID for power button #85

Merged
merged 1 commit into from
Sep 25, 2024

Conversation

tari
Copy link

@tari tari commented Sep 22, 2024

Setting an ID for the power button allows its behavior to be extended or modified by configuration that uses this package, for example:

packages:
  Athom_Technology.Smart_Plug_V2: github://athom-tech/athom-configs/athom-smart-plug-v2.yaml

binary_sensor:
  - id: !extend power_button
    # Additional settings here

@Roving-Ronin
Copy link

If your wanting to get this added in, I would suggest that you also add this into the:

v1 Plug: https://github.com/athom-tech/athom-configs/blob/main/athom-smart-plug.yaml

v3 Plug: https://github.com/athom-tech/esp32-configs/blob/main/athom-smart-plug.yaml

to ensure consistency of the functionality / code between ALL versions of the plug. Previously this was lacking, but in the last few months was just fixed up.

Setting an ID for the power button allows its behavior to be extended or
modified by configuration that uses this package, for example:

    packages:
      Athom_Technology.Smart_Plug_V2: github://athom-tech/athom-configs/athom-smart-plug-v2.yaml

    binary_sensor:
      - id: !extend power_button
        # Additional settings here
@tari tari changed the title smart-plug-v2: add id for power button Smart plugs: add ID for power button Sep 22, 2024
@tari
Copy link
Author

tari commented Sep 22, 2024

That's reasonable. Added the same change to v1 here, and v3 in athom-tech/esp32-configs#27.

@tarontop tarontop merged commit 207a0ac into athom-tech:main Sep 25, 2024
22 checks passed
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.

3 participants