Fixes
```
Nov 08 21:41:14 homeassistant homeassistant[474]: /usr/src/homeassistant/homeassistant/components/bluetooth/util.py:39: FutureWarning: BLEDevice.rssi is deprecated and will be removed in a future version of Bleak, use AdvertisementData.rssi instead
Nov 08 21:41:14 homeassistant homeassistant[474]: rssi=history.device.rssi,
```
* Add Picnic services for searching products and adding products to the cart
* Improve the Picnic services implementation and add unit tests
* Fix pre-commit check issues
* Fix comments and example product name
* Remove search service, update add_product service schema
* Fix pylint suggestion
* Add more tests and removed unused code
* Remove code needed for the removed service, clean tests from obvious comments and add type hints
* Remove unused import
* Remove unnecessary comments and simplify getting the config entry id
Co-authored-by: Allen Porter <allen.porter@gmail.com>
* Don't use hass.data in tests, make device id mandatory for service
* Rewrite all service tests so using lru.cache is not needed
* Add test for uncovered line in _product_search()
* Require a config entry id as service parameter instead of device id
* Use explicit check in get_api_client() and raise HomeAssistantError
* Fix HomeAssistantError import, fix services tests
* Change HomeAssistantError to ValueError when config entry is not found
Co-authored-by: Allen Porter <allen.porter@gmail.com>
* Add option to include attachments from remote URL to Discord notifications
* Use aiohttp instead of requests for Discord Notify URL retrieval
* Refactor discord notifications code
* Remove unecessary images check in discord notifier
* Implemented RestoreEntity
Merged commit conflict
* removed accidental change
* Update homeassistant/components/dynalite/dynalitebase.py
Co-authored-by: Erik Montnemery <erik@montnemery.com>
* added tests for the state
* added tests for switch state
* moved to ATTR_x and STATE_x instead of strings
some fixes to test_cover
* moved blind to DEVICE_CLASS_BLIND
* used correct constant instead of deprecated
* Implemented RestoreEntity
* removed accidental change
* added tests for the state
* added tests for switch state
* moved to ATTR_x and STATE_x instead of strings
some fixes to test_cover
* fixed isort issue from merge
Co-authored-by: Erik Montnemery <erik@montnemery.com>
* Fix accelerator sensor in fibaro integration
* Implement suggestions from code review
* Implement suggestions from code review
* Changes as suggested in code review
* Adjust as suggested in code review
This is a small refactor to align the bluetooth scanner class
to the same design used in the esphome scanner to pass the
callback for new service infos in the constructor.
The original thinking was that multiple callbacks would
be needed to send the service infos to different paths,
however with the creation of a central manager, this
designed turned out to not be needed and can be
simplified