Open alexyao2015 opened 2 months ago
The get_service
call just got converted to an async function. https://github.com/home-assistant/core/pull/116068
Its not thread-safe to call hass.http.register_view
or hass.services.async_register
in the executor. It should have been an an async function all along
Its not thread-safe to call
hass.http.register_view
orhass.services.async_register
in the executor. It should have been an an async function all along
Ah ok, my bad I didn't know that, thanks @bdraco. This is all good from me, thanks @alexyao2015!
testing....
dismiss service is there
notify.html5 service is missing though
Issue is fixed
I did some testing. Some observations:
Breaking change
Proposed change
This converts HTML5 to use config flow. In the process, this also removes the need for the user to manually obtain a VAPID key as it is now automatically generated.
This also removes the long deprecated GCM configuration method.
Type of change
Additional information
Checklist
ruff format homeassistant tests
)If user exposed functionality or configuration variables are added/changed:
If the code communicates with devices, web services, or third-party tools:
Updated and included derived files by running:
python3 -m script.hassfest
.requirements_all.txt
.Updated by running
python3 -m script.gen_requirements_all
..coveragerc
.To help with the load of incoming pull requests: