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

Feat: Reconfiguration config flow #434

Open
wants to merge 8 commits into
base: master
Choose a base branch
from

Conversation

bj00rn
Copy link
Contributor

@bj00rn bj00rn commented Oct 30, 2024

DON'T MERGE before #431

Add reconfiguration flow

fixes #102

Requires some more testing.. Based on #431

  • Some refactoring of config flow helpers
  • Translations for SV and EN for reconfigure_confirm
image

@@ -10,12 +10,6 @@ if [[ ! -d "${PWD}/config" ]]; then
hass --config "${PWD}/config" --script ensure_config
fi

Copy link
Collaborator

Choose a reason for hiding this comment

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

Why was this removed?

Copy link
Contributor Author

@bj00rn bj00rn Nov 8, 2024

Choose a reason for hiding this comment

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

This change is from PR #431. I've replaced usage of pythonpath with mounting custom_components in the devcontainer. Downside is that this only works inside the devcontainer.

It seems that setting adding integration to pythonpath will cause the integration package to shadow nordpool dependency, since pythonpath has precedence over site-packages. Integration will not load. See discussion here ludeeus/integration_blueprint#130

import nordpool will point to the integration itself. I don't now if/why this was working in the earlier version of the devcontainer. Maybe some internal has changed in HASS, or if this relates to changes in python 3.12

@Hellowlol
Copy link
Collaborator

This looks very nice, i have not been able to test it so it will not make the new release for today but this is a really nice feature. I will review and merge in 0.0.17 :)

@bj00rn
Copy link
Contributor Author

bj00rn commented Nov 8, 2024

This looks very nice, i have not been able to test it so it will not make the new release for today but this is a really nice feature. I will review and merge in 0.0.17 :)

This PR is based on #431 since reconfigure flow is a 2024.4 feature. It will have to be rebased when #431 is merged.

I think this PR needs some testing before it can be merged. Especially the entity naming. A new entity will be created when changing settings since named is derived from VAT, currency etc

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.

Make sensor settings editable (OptionHandler)
2 participants