Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Impossible to configure the integration settings #18

Closed
chlongv opened this issue Dec 18, 2023 · 2 comments
Closed

Impossible to configure the integration settings #18

chlongv opened this issue Dec 18, 2023 · 2 comments
Labels
duplicate This issue or pull request already exists

Comments

@chlongv
Copy link
Contributor

chlongv commented Dec 18, 2023

Version of the custom_component

1.5.0

Configuration

No configuration yet, as I have not been able to reach this point.

Describe the bug

I'm not able to add the solar_optimizer integration. I'm not able to validate the initial configuration, the sell_cost_entity_id and buy_cost_entity_id fields are never valid.

image

Also, I get no suggestion for the sell_cost_entity_id and buy_cost_entity_id fields:

image

This is maybe linked to #15 where the schema for these 2 fields has been changed ?

Debug log


2023-12-18 22:06:15.970 WARNING (SyncWorker_1) [homeassistant.loader] We found a custom integration solar_optimizer which has not been tested by Home Assistant. This component might cause stability problems, be sure to disable it if you experience issues with Home Assistant
2023-12-18 22:06:15.970 WARNING (SyncWorker_1) [homeassistant.loader] We found a custom integration hacs which has not been tested by Home Assistant. This component might cause stability problems, be sure to disable it if you experience issues with Home Assistant
2023-12-18 22:08:50.971 DEBUG (MainThread) [custom_components.solar_optimizer.config_flow] config_flow step user (1). 1er appel : pas de user_input -> on affiche le form user_form


@jmcollin78
Copy link
Owner

Hello as said in the error, thoses fields need to be some input_number.
Capture d’écran 2023-12-18 à 23 12 59

Sensors are not accepted yet but there is many requests to add it: #15

The PR is merged but not released yet due to a lack of time. If you need this rapidly, you can install the "main" branch. You have to check the "use unstable release" (something like that) on HACS to be able to install it.

I hope this helps.

@jmcollin78 jmcollin78 added the duplicate This issue or pull request already exists label Dec 18, 2023
@chlongv
Copy link
Contributor Author

chlongv commented Dec 19, 2023

Thanks for the quick answer. Sorry for the duplicate, I hadn't realized #15 hadn't been released yet and that input_number is an integration and not a "plain value".

I'm probably going to use the "main" branch as I already have sensors that give the correct buy and sell tariffs.

@chlongv chlongv closed this as completed Dec 19, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
duplicate This issue or pull request already exists
Projects
None yet
Development

No branches or pull requests

2 participants