#Hey guys. I'm new here, trying to get an
shut mason
I looked at a few ones and they seem to be "old style"?
Not matching what scaffold produces initially
Am I looking at the wrong ones somehow?
Maybe just point me to a good recent one which implements a platform.
pale fractal
Like what are you looking for
like if you tell me with what you struggle I probably know some integrations to look at
shut mason
Right now I was looking into how config flows work (https://developers.home-assistant.io/docs/config_entries_config_flow_handler/) but I would like to see a fully working example including the GUI which it produces.
Ideally I'd find an integration similar to what I'm trying to do (a platform for notify)
pale fractal
I dont think there are a lot of
there are some
I have a spreadsheet somewhere
nvm
It doesnt have notify
But to check a config flow, well there are quite some for that
you just seem to have a basic config flow
Checkout Suez Water, I implemented that recently
shut mason
Ok great, I'll check it out. I will install it in a test HA environment to see what it looks like and look at the code in parallel.
Thanks.
shut mason
Are you sure it's possible to use a config flow for a platform while extending the PLATFORM_SCHEMA from homeassistant.components.notify? I found 38 integratoins that use and extend that schema. Only 2 of those have a config flow and it seems they use the config flow only for the other integration parts they provide, not for notify.
It's just a bit hard for me to fulfil requirements that no existing notify integration seems to fulfil, since I have no example to look at.
shut mason
Oh I see
pale fractal
all new integrations (like if you integrate with a device or service) should use config flow
protocols like MQTT and modbus are an exception
shut mason
I'll try to find a notify-only integration which uses configflow.
pale fractal
But config flow is not unique to notify
System bridge might be an example
But it has more platforms
But you just have to apply it
shut mason
Ok, I think I managed finally. I updated the PR. See https://github.com/home-assistant/core/pull/105977
Thanks for your help.
pale fractal
you still need to run hassfest
shut mason
I did:
$ python3 -m script.hassfest
Validating application_credentials... done in 0.06s
Validating bluetooth... done in 0.01s
Validating codeowners... done in 0.03s
Validating config_schema... done in 1.26s
Validating dependencies... done in 2.68s
Validating dhcp... done in 0.01s
Validating json... done in 0.00s
Validating manifest... done in 0.15s
Validating mqtt... done in 0.02s
Validating requirements... done in 0.02s
Validating services... done in 2.57s
Validating ssdp... done in 0.01s
Validating translations... done in 5.15s
Validating usb... done in 0.01s
Validating zeroconf... done in 0.01s
Validating config_flow... done in 0.12s
Validating coverage... done in 0.09s
Validating docker... done in 0.00s
Validating mypy_config... done in 0.04s
Validating metadata... done in 0.01s
Integrations: 1222
Invalid integrations: 0
intersting
because you do have tests but hassfest is not adding you to the codeowners of test
shut mason
I see. Well I don't know why that is..
I can try to debug it tomorrow or so.
I can also add a line manually in CODEOWNERS, can I?
shut mason
🙏