Skip to content

add new features to Heiman HS1SA-E#11239

Merged
Koenkk merged 49 commits intoKoenkk:masterfrom
Leo2442926161:master
Jan 12, 2026
Merged

add new features to Heiman HS1SA-E#11239
Koenkk merged 49 commits intoKoenkk:masterfrom
Leo2442926161:master

Conversation

@Leo2442926161
Copy link
Copy Markdown
Contributor

Hi Z2M team,
I am adding new features to Heiman HS1SA-E, could you please have an review on it. thanks.

Comment thread src/devices/heiman.ts Outdated
0: "muted", // bit0
1: "alarm muted", // bit1
2: "fault muted", // bit2
3: "low battery muted", // bit3
Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

Please keep _ instead of , also above in faultStateBitMap

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Hi @Koenkk , Thanks for point out, but interestingly, when I put underline there, the entities are exposed with _ to HA, too, that looked like alarm_muted instead of alarm mute, which looks weird. is there any ways to correct it?

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

Yes that's expected (and how it's done for all devices)

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Yes that's expected (and how it's done for all devices)

thanks, fixed, but still wonder why.

@Leo2442926161 Leo2442926161 requested a review from Koenkk January 12, 2026 04:23
@Koenkk Koenkk merged commit fb6dfd7 into Koenkk:master Jan 12, 2026
3 checks passed
@Koenkk
Copy link
Copy Markdown
Owner

Koenkk commented Jan 12, 2026

thanks

Koenkk added a commit that referenced this pull request Feb 3, 2026
Co-authored-by: Koen Kanters <[email protected]>
Co-authored-by: autofix-ci[bot] <114827586+autofix-ci[bot]@users.noreply.github.com>
Co-authored-by: Leo <[email protected]>
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