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

add armed night #24

Merged
merged 2 commits into from
Jul 26, 2024
Merged

add armed night #24

merged 2 commits into from
Jul 26, 2024

Conversation

Aidenkwok
Copy link
Contributor

map partset_b on HKC to arm_night on HA

@github-actions github-actions bot closed this Jul 22, 2024
@jasonmadigan jasonmadigan reopened this Jul 25, 2024
Repository owner deleted a comment from github-actions bot Jul 25, 2024
@jasonmadigan jasonmadigan self-assigned this Jul 25, 2024
@jasonmadigan
Copy link
Owner

Think we also need:

class HKCAlarmControlPanel(AlarmControlPanelEntity, CoordinatorEntity):
    _attr_supported_features = (
        AlarmControlPanelEntityFeature.ARM_HOME
        | AlarmControlPanelEntityFeature.ARM_AWAY
        | AlarmControlPanelEntityFeature.ARM_NIGHT
    )

@Aidenkwok
Copy link
Contributor Author

_attr_supported_features = (

added 👍

@jasonmadigan
Copy link
Owner

thanks @Aidenkwok!

@jasonmadigan jasonmadigan merged commit a219190 into jasonmadigan:main Jul 26, 2024
2 checks passed
@jasonmadigan
Copy link
Owner

I'll create a pre-release at the weekend to test.

@jasonmadigan
Copy link
Owner

@Aidenkwok https://github.com/jasonmadigan/ha-hkc/releases/tag/1.0.17-rc2 - can check it out via "beta" releases in HACS. My partset B is set to "panic", but it does seem to work. You might give it a spin too. If happy, will push final.

@Aidenkwok
Copy link
Contributor Author

@jasonmadigan can confirm its working for me too!

@jasonmadigan
Copy link
Owner

excellent, thanks for the contrib @Aidenkwok

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.

2 participants