Remove Flick Electric integration#155469
Conversation
|
Hey there @joostlek, mind taking a look at this pull request as it has been labeled with an integration ( Code owner commandsCode owners of
|
| @@ -928,7 +929,6 @@ | |||
| "oncue": 73, | |||
| "tailwind": 44, | |||
| "dunehd": 102, | |||
| "flick_electric": 42, | |||
| "home_plus_control": 85, | |||
There was a problem hiding this comment.
Not sure if this was actually needed to be removed - looks like it's just snapshot of analytics data where the integration so maybe not?
There was a problem hiding this comment.
Pull Request Overview
This PR removes the Flick Electric integration from Home Assistant. According to the PR description, all Flick Electric customers have been migrated to Meridian Energy, and the API endpoint https://api.flick.energy now returns NXDOMAIN, making the integration non-functional.
- Complete removal of the
flick_electricintegration source code and supporting files - Removal of all tests, fixtures, and test configuration
- Cleanup of generated files and metadata references
Reviewed Changes
Copilot reviewed 19 out of 21 changed files in this pull request and generated no comments.
Show a summary per file
| File | Description |
|---|---|
| homeassistant/components/flick_electric/* | Removed all integration source files (manifest, sensor, config flow, coordinator, constants, strings) |
| tests/components/flick_electric/* | Removed all test files, fixtures, and test configuration |
| homeassistant/generated/config_flows.py | Removed flick_electric from config flows registry |
| homeassistant/generated/integrations.json | Removed integration metadata entry |
| requirements_all.txt | Removed PyFlick==1.1.3 dependency |
| requirements_test_all.txt | Removed PyFlick==1.1.3 test dependency |
| script/hassfest/quality_scale.py | Removed flick_electric from quality scale exemption lists |
| tests/components/analytics_insights/fixtures/current_data.json | Removed analytics entry for the integration |
| CODEOWNERS | Removed @ZephireNZ ownership entries |
1276cdd to
791cbb1
Compare
|
For context, I found this RNZ article about the sale of Flick Electric to Meridian: https://www.rnz.co.nz/news/business/560899/what-went-wrong-for-flick |
Breaking change
The
flick_electricintegration has been removed. All customers of the Flick Electric company have already been moved to Meridian Energy. The API endpoint used in the integration is already non functional.Proposed change
Removes the
flick_electricintegration from Home Assistant.All customers have been moved to Meridian Energy, to my knowledge there is not an official HA integration which these users could use instead.
The integration will be non-functional, as the API endpoint
https://api.flick.energynow returnsNXDOMAIN.Type of change
Additional information
Checklist
ruff format homeassistant tests)Tests have been added to verify that the new code works.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.To help with the load of incoming pull requests: