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

Enhance homeassistant discovery 2 #1274

Closed
wants to merge 9 commits into from
Closed

Enhance homeassistant discovery 2 #1274

wants to merge 9 commits into from

Conversation

caco3
Copy link
Collaborator

@caco3 caco3 commented Nov 4, 2022

re-creation of #1246

  • 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 and others added 9 commits November 4, 2022 17:32
@caco3
Cleanup REST API (#1255)
f3e73ec 
* Replaced URIs:
- value.html => value
- statusflow.html => statusflow
- cputemp.html => cputemp
- rssi.html => rssi
- statusflow.html => statusflow

Removed URLs:
 - wasserzaehler.html

* keep legacy API

* .

* .

* .

* .

* .

* .

* updated links
@jomjol
Remove ErrorMessage
4ffde22
@bzfbd
Fix various warnings which become fatal with later gcc versons in esp…
235861e 
…-idf 5.x (#1268)

- we cannot use partial initialisation of structs in C++ files (copied from example C files initially it seems)
- IRAM_ATTR uses a COUNTER, do not use the attribute on the implementation
- provide missing copy implementations for Rgb and Hsv
- one no longer can |= on volatile variables; use = | instead
- fix project and header includes
- avoid redefining BLINK_GPIO
- Remove defined but unused variables
- Fix printf formats
- Add missing case statement (HTTP_EVENT_REDIRECT)
- RMT needs to be updated to new interface (CONFIG_RMT_SUPPRESS_DEPRECATE_WARN is on currently; see https://docs.espressif.com/projects/esp-idf/en/release-v5.0/esp32/api-reference/peripherals/rmt.html)
- Adjust tcpip_adpater_* to esp_netif_*
- Use buffered versions of *ntoa* functions for IPv4 addresses and not a static on the stack (also fixes warnings)
- Whatever I missed

Co-authored-by: Bjoern A. Zeeb <patch@zabbadoz.net>
@bzfbd
Correct spelling of "Hostname" (#1270)
16e0cd3 
Co-authored-by: Bjoern A. Zeeb <patch@zabbadoz.net>
@jomjol
Correct sdkonfig
3b31cbb
@jomjol
Increase max handler due to new handlers
cbd63ad
@caco3
Merge branch 'rolling' of https://github.com/jomjol/AI-on-the-edge-de…
a44594d 
…vice into rolling
@caco3
Merge branch 'rolling' of https://github.com/jomjol/AI-on-the-edge-de…
1ab89f8 
…vice into rolling
@caco3
manual re-creation of #1246
83070d9
@caco3 caco3 closed this Nov 4, 2022
@caco3 caco3 deleted the enhance-homeassistant-discovery-2 branch November 7, 2022 15:32
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.
3 participants
@caco3 @jomjol @bzfbd