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

README.md - Add note to access frontend UI #884

Merged
merged 1 commit into from
Dec 19, 2023

Conversation

baylanger
Copy link
Contributor

I'm creating this PR simply because I wasn't sure if the adaptive_lighting: entry is a requirement to access the UI. The old Configuration -> Integrations now named Settings -> Devices & Services confused me.

After reading this below, I'm unclear if it's possible to add configuration entries in the YAML and others via the UI. I'm used to see integrations either use either YAML or UI but never both at the same time. Per the wordings, it seems like it's possible to use both at the same time but likely making sure there's no collision, i.e. 2 identical names.

Configure an Adaptive Lighting component. Option names align with the YAML settings. If you've defined this entry in YAML, no options will appear here. For interactive graphs that demonstrate parameter effects, visit [this web app](https://basnijholt.github.io/adaptive-lighting). For further details, see the [official documentation](https://github.com/basnijholt/adaptive-lighting#readme).

Per any comments, the wording will probably need some adjustments but feel free to close this PR and make the proposed changes via another one.

@basnijholt basnijholt merged commit 1601a22 into basnijholt:main Dec 19, 2023
1 check passed
@basnijholt
Copy link
Owner

It's possible to have both yes.

Thanks for fixing this!

@all-contributors please add @baylanger for docs

Copy link
Contributor

@basnijholt

I couldn't determine any contributions to add, did you specify any contributions?
Please make sure to use valid contribution names.

I've put up a pull request to add @baylanger! 🎉

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.

2 participants