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

Extend Homeassistant discovery with button for flow start #3415

Merged
merged 5 commits into from
Dec 4, 2024

Conversation

caco3
Copy link
Collaborator

@caco3 caco3 commented Dec 1, 2024

ctrl/flow_start

Adds a button to to Homeassistant to trigger the flow:
grafik

out of scope for this Pull request:

@caco3 caco3 marked this pull request as draft December 1, 2024 23:19
@caco3 caco3 changed the title Extend HomeAssistant discovery Extend Homeassistant discovery Dec 1, 2024
@caco3
Copy link
Collaborator Author

caco3 commented Dec 2, 2024

@Slider0007 I tested your code for the flow control start and it seems to work well.
I looked into how to do it with the set_prevalue, how ever the problem seems to be that we can't pre-configure HA to show a useful input. I tried it with a homeassistant/number/ (see #3416), how ever the shown slider is not useful at all:
grafik

See also https://community.home-assistant.io/t/mqtt-number-optional-mode-for-slider-box/362880

One would have to implement it with 2 elements like shown here: https://community.home-assistant.io/t/prompting-for-input-in-a-script/602702/4. But i doubt this is possible with the discovery protocol.

Therefore I would keep the set_prevalue out of scope.

The GPIOs should be feasible, but I haven't looked into it yet. Ideally on the HA UI they would represent the real state.

@caco3 caco3 changed the title Extend Homeassistant discovery Extend Homeassistant discovery with button for flow start Dec 4, 2024
@caco3 caco3 marked this pull request as ready for review December 4, 2024 22:16
@caco3 caco3 merged commit 179005f into main Dec 4, 2024
12 checks passed
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