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.

Sign up for GitHub

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

Jump to bottom

Fix HA naming issue #19

Draft
wants to merge 1 commit into
base: main
Choose a base branch
from
Draft

Fix HA naming issue #19

wants to merge 1 commit into from

Conversation

awilliams
Copy link
Owner

@awilliams awilliams commented Oct 22, 2023 •
edited
Loading

Fixes the issue where HA complains that

MQTT entity name starts with the device name in your config

Before, the entity was named: $device_name $ap_name. Now, it's named: $device_mac $ap_name.

Fixes #18

@awilliams
Change entity name to not include device name
4ddf47c 
This seems to fix the new HA error of:
> MQTT entity name starts with the device name in your config

Also:
The Homeassistant setup needs updating for newer version. This allows
for a basic instance of HASS to be run via docker compose, and then
for wifi-presence to send events to it via MQTT.

Fixes #18
@ThatcherC
Copy link

ThatcherC commented Nov 17, 2023 •
edited
Loading

How's this MR looking? I'd like to try out wifi-presence on my router and it looks like this newest version would be the best one to try. Code changes look solid to me!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

MQTT entity name starts with the device name
2 participants
@awilliams @ThatcherC