Skip to content

Conversation

dm82m
Copy link
Contributor

@dm82m dm82m commented Mar 11, 2025

first draft

@andig
Copy link
Member

andig commented Mar 11, 2025

We don‘t plan to merge this. Yaml is on its way to retirement.

@andig andig closed this Mar 11, 2025
@dm82m
Copy link
Contributor Author

dm82m commented Mar 11, 2025

@naltatis do you see any chance to provide this possibility also in ui config? providing a syntax for env vars so that they get replaced by corresponding env vars during runtime?

@naltatis
Copy link
Member

@dm82m now sure how this is supposed to work. I don't see, that we add env interpolation support to each field in config ui. There are still places where we do yaml today (tariffs, ...) but we'll also migrate these to proper guided forms. I guess we need another solution.
What's the goal? Not having to repeat credentials?

@dm82m
Copy link
Contributor Author

dm82m commented Mar 11, 2025

Within the evcc home assistant addon we can use a proxy to access home assistant api. With this we can get data from home assistant into evcc. I.e. car soc, car range etc.
To do that we need a bearer token that is available within the container as env var. it would be great to be able to use it in evcc.

As andi declined my yaml modification, which I can understand, the only way would be to add it in Ui config, but I am not skilled enough to find a proper way there.

see evcc-io/hassio-addon#142

@GrimmiMeloni
Copy link
Collaborator

I think this is somewhat "poweruser" space, so maybe we could do without interpolation inside the UI.
Isn't conceptually the DB just another storage (just like the YAML on the filesystem)?
The idea would be that independent of where the config comes from we could replace the vars as suggested by the PR.

@dm82m
Copy link
Contributor Author

dm82m commented Mar 12, 2025

Fine for me, but I have technically no idea how that could be solved 😂

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.

4 participants