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

### DELETE ### Enhance Homeassistant Discovery #1246

Closed
wants to merge 66 commits into from

Conversation

caco3
Copy link
Collaborator

@caco3 caco3 commented Oct 30, 2022

  • Added parameter to enable/disable discovery
  • Added parameter to select meter type. This is used to set the device_class and unit_of_meas correctly
  • Corrected various Discovery parameters, use diagnostic in the entity_category for the diagnostic sensors

New Topics

  • interval: Interval of the rounds in minutes as configured in the config.

Notes

  • The Discovery Topics and the static topics (IP, MAC, Hostname, ...) only get sent on startup. To trigger a re-sending of them, you can call the following URL: http://<IP>/mqtt_publish_discovery

@caco3 caco3 mentioned this pull request Oct 31, 2022
3 tasks
@caco3 caco3 marked this pull request as ready for review October 31, 2022 23:03
@caco3 caco3 marked this pull request as draft October 31, 2022 23:04
@caco3
Copy link
Collaborator Author

caco3 commented Oct 31, 2022

Waiting for feedback from #1133

@caco3 caco3 mentioned this pull request Nov 2, 2022
caco3 added a commit that referenced this pull request Nov 4, 2022
caco3 added a commit that referenced this pull request Nov 4, 2022
@caco3 caco3 changed the title Enhance Homeassistant Discovery ### DELETE ### Enhance Homeassistant Discovery Nov 4, 2022
@caco3 caco3 closed this Nov 4, 2022
jomjol pushed a commit that referenced this pull request Nov 5, 2022
* manual re-creation of #1246

* removed non-working approach to get interval

* postpone the MQTT service start until the ClassFlowControll::ReadParameter() gets called

* Increase the max_uri_handlers to 35 (was 30)

* removed newlines in logs

* added parameter to UI

* Register handler to re-start MQTT Discovery

* fix param
@caco3 caco3 deleted the enhance-homeassistant-discovery branch November 7, 2022 15:32
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.

1 participant