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

Major rework of Home Assistant MQTT config and additional info #515

Merged
merged 10 commits into from
Oct 27, 2022

Conversation

duhow
Copy link
Contributor

@duhow duhow commented Oct 23, 2022

  • Use template functions to define components with custom content (sensor and switch)
  • Better readability and easy of adding new components to Home Assistant
  • Add entity_category to organize components into diagnostic (system info), config (switches) and default (image, camera and sensors).
  • Add connections section to identify device via MAC address
  • Reduce mqtt_adv_homeassistant.sh size about 10kb

@duhow duhow changed the title Major rework of Home Assistant MQTT Configuration and entity_category Major rework of Home Assistant MQTT config and additional info Oct 23, 2022
@roleoroleo
Copy link
Owner

I'm not able to test this PR because I can't "destroy" my ha configuration.
My wife would not agree :)
If you've tested it I'll merge.

@duhow
Copy link
Contributor Author

duhow commented Oct 26, 2022

Sure, I have been testing this locally with my camera. This is the end result:

image

@duhow
Copy link
Contributor Author

duhow commented Oct 26, 2022

Depending on the status, the device has to be deleted from Home Assisstant MQTT and then reconfigured (keeps same identifiers) to repopulate all fields accordingly.

@roleoroleo
Copy link
Owner

Thank you for your code.

@roleoroleo roleoroleo merged commit ee0e3a8 into roleoroleo:master Oct 27, 2022
@roleoroleo
Copy link
Owner

@duhow
Do you agree if I copy your changes to the other projects (Allwinner and MStar)?

@duhow
Copy link
Contributor Author

duhow commented Nov 18, 2022

Sure @roleoroleo ! 👍🏻

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