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(ui): allow to toggle persistent/discovery fields from HA discovery table #3569

Merged
merged 2 commits into from
Jan 29, 2024

Conversation

robertsLando
Copy link
Member

Fixes #3558

@robertsLando
Copy link
Member Author

2024-01-29_09-56

@coveralls
Copy link

Pull Request Test Coverage Report for Build 7693488676

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage remained the same at 22.117%

Totals Coverage Status
Change from base Build 7655652062: 0.0%
Covered Lines: 3763
Relevant Lines: 18090

💛 - Coveralls

@robertsLando robertsLando merged commit b86b84c into master Jan 29, 2024
9 checks passed
@robertsLando robertsLando deleted the toggle-hass-discovery branch January 29, 2024 08:59
@apella12
Copy link
Contributor

Awesome ! Tested a bit this am and it's GREAT.

I need to familiarize myself with how this works to be more efficient. What I did was disable the HA ids that I did not want, hit store (makes them persistent on Startup?). Go to MQTT Explorer to delete the HA topics. Restart ZUI to make sure, then restart OH to clear the unwanted topics.

Was thinking the best way for a new installation was to set HA to Manual, edit the HA ids in the nodes, then switch back to Auto. Does that make sense?

Anyway, thanks again.

@robertsLando
Copy link
Member Author

Was thinking the best way for a new installation was to set HA to Manual, edit the HA ids in the nodes, then switch back to Auto. Does that make sense?

Exactly that's why the manual discovery exists :) Glad you like this addition, wondering why I didn't did it before now 😆

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.

Blacklist some (but not all) HA config discoveries
3 participants