Skip to content

Add config flow (Settings > Integrations)#349

Open
teh-hippo wants to merge 3 commits intomacxq:mainfrom
teh-hippo:pr/ui-config-flow
Open

Add config flow (Settings > Integrations)#349
teh-hippo wants to merge 3 commits intomacxq:mainfrom
teh-hippo:pr/ui-config-flow

Conversation

@teh-hippo
Copy link
Copy Markdown

Adds config flow support so the integration can be set up through the Home Assistant UI instead of requiring configuration.yaml. Related to #199.

The setup basically mimics the YAML inputs, and does some checks before accepting them.

If a YAML platform config is already active, the flow aborts with a message asking the user to remove it first. Existing YAML setups continue to work without changes.

Depends on #348 (solar-only support).

Tested end-to-end on a live HA instance.

Skip battery entity creation when the FoxESS API reports
hasBattery=false. Adds optional hasBattery config override.
Reduces log noise for solar-only systems going offline at night.
@teh-hippo teh-hippo force-pushed the pr/ui-config-flow branch from 126636a to 6ac2383 Compare March 3, 2026 13:35
The integration uses the FoxESS Cloud API (foxesscloud.com),
not local device communication.
Add config flow so the integration can be set up via the HA UI.
The setup step collects API key, device serial number, and name,
then validates against the FoxESS Cloud API.

Battery status is auto-detected from the API response and exposed
as an option toggle alongside extended PV and Evo inverter mode.

Existing YAML configurations continue to work unchanged.
@teh-hippo teh-hippo force-pushed the pr/ui-config-flow branch from 6ac2383 to 517f36e Compare March 3, 2026 13:36
@teh-hippo
Copy link
Copy Markdown
Author

@macxq would you be able to cast your eye over this?

Copy link
Copy Markdown

@svanscho svanscho left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM 🚀 nice changes

"dependencies": ["rest"],
"documentation": "https://github.com/macxq/foxess-ha",
"iot_class": "local_polling",
"iot_class": "cloud_polling",
Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice one 👍

"domain": "foxess",
"name": "HA & FoxESSCloud integration",
"codeowners": ["@macxq","@r-amado","@fozzieuk"],
"config_flow": true,
Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🥳

@danelphick
Copy link
Copy Markdown

I just found this after submitting my own config_flow PR #369. Looks like there's demand for such a feature!

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.

3 participants