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

Add visibility and color parameters to UI sitemap provider #3075

Merged
merged 4 commits into from
Sep 6, 2022

Conversation

mherwege
Copy link
Contributor

@mherwege mherwege commented Sep 1, 2022

This adds the missing parameters to the sitemap provider. This allows the UI to be further extended in a follow-up PR and make the sitemap UI configuration support the same features as textual configuration.

Signed-off-by: Mark Herwege mark.herwege@telenet.be

Signed-off-by: Mark Herwege <mark.herwege@telenet.be>
@openhab-bot
Copy link
Collaborator

This pull request has been mentioned on openHAB Community. There might be relevant details there:

https://community.openhab.org/t/sitemap-future-considerations/138517/9

@J-N-K J-N-K added the bug An unexpected problem or unintended behavior of the Core label Sep 3, 2022
Signed-off-by: Mark Herwege <mark.herwege@telenet.be>
@mherwege mherwege requested a review from J-N-K September 5, 2022 12:23
Copy link
Member

@J-N-K J-N-K left a comment

Choose a reason for hiding this comment

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

Otherwise LGTM, thanks.

Signed-off-by: Mark Herwege <mark.herwege@telenet.be>
@mherwege mherwege requested a review from J-N-K September 6, 2022 06:13
Signed-off-by: Mark Herwege <mark.herwege@telenet.be>
@J-N-K
Copy link
Member

J-N-K commented Sep 6, 2022

Thanks!

@J-N-K J-N-K added enhancement An enhancement or new feature of the Core and removed bug An unexpected problem or unintended behavior of the Core labels Sep 6, 2022
@J-N-K J-N-K added this to the 3.4 milestone Sep 6, 2022
@J-N-K J-N-K merged commit c61ea77 into openhab:main Sep 6, 2022
@mherwege mherwege deleted the sitemap branch September 13, 2022 07:08
splatch pushed a commit to ConnectorIO/copybara-hab-core that referenced this pull request Jul 12, 2023
)

* add visibility and color parameters

Signed-off-by: Mark Herwege <mark.herwege@telenet.be>
GitOrigin-RevId: c61ea77
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement An enhancement or new feature of the Core
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants