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

fix: WARNING in HA => MQTT entity name starts with the device name in… #327

Merged
merged 1 commit into from Feb 3, 2024

Conversation

Minims
Copy link
Contributor

@Minims Minims commented Jan 26, 2024

This PR fix warnings in HomeAssistant since MQTT entity name has changed:

WARNING (MainThread) [homeassistant.components.mqtt.mixins] MQTT entity name starts with the device name in your config {'availability_topic': 'yi/yi-home-720p/status', 'payload_available': 'online', 'payload_not_available': 'offline', 'device': {'identifiers': ['yi-home-720p'], 'manufacturer': 'Yi', 'model': 'Yi Home 720p', 'name': 'Yi Home 720p', 'sw_version': '0.4.1d', 'connections': []}, 'qos': 1, 'device_class': <BinarySensorDeviceClass.MOTION: 'motion'>, 'state_topic': 'yi/yi-home-720p/ai_human_detection', 'name': 'Yi Home 720p Human Detection', 'unique_id': 'yi-home-720p-ai_human_detection', 'payload_on': 'human', 'off_delay': 60, 'force_update': False, 'enabled_by_default': True, 'payload_off': 'OFF', 'availability_mode': 'latest', 'encoding': 'utf-8'}, this is not expected. Please correct your configuration. The device name prefix will be stripped off the entity name and becomes 'Human Detection'

@Minims
Copy link
Contributor Author

Minims commented Jan 31, 2024

@alienatedsec Can we add this in the pre release too ?

@alienatedsec
Copy link
Owner

@Minims I will do once I have the other merge conflicts sorted

@alienatedsec
Copy link
Owner

@Minims This will need to be merged later unless you raise a pull request into a pre-release https://github.com/alienatedsec/yi-hack-v5/tree/pre-release-0.4.1x

@Minims Minims changed the base branch from master to pre-release-0.4.1x February 3, 2024 10:58
@Minims
Copy link
Contributor Author

Minims commented Feb 3, 2024

@alienatedsec I have update the destination of the PR to pre-release-0.4.1x

@alienatedsec
Copy link
Owner

@roleoroleo please can you have a quick look?

@roleoroleo
Copy link
Collaborator

@roleoroleo please can you have a quick look?

The PR is ok.

@alienatedsec alienatedsec merged commit ce95a00 into alienatedsec:pre-release-0.4.1x Feb 3, 2024
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.

None yet

3 participants