Skip to content

Update thing-description schema again#513

Merged
stefan-hoehn merged 1 commit intoopenhab:mainfrom
andrewfg:update-schema-again
Apr 23, 2025
Merged

Update thing-description schema again#513
stefan-hoehn merged 1 commit intoopenhab:mainfrom
andrewfg:update-schema-again

Conversation

@andrewfg
Copy link
Contributor

Depends on openhab/openhab-core#4738

Signed-off-by: Andrew Fiddian-Green software@whitebear.ch

Signed-off-by: Andrew Fiddian-Green <software@whitebear.ch>
@andrewfg
Copy link
Contributor Author

Pinging @lsiepel / @jlaur

Copy link

@lsiepel lsiepel left a comment

Choose a reason for hiding this comment

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

Yes, matches openhab/openhab-core#4738. LGTM

@lsiepel
Copy link

lsiepel commented Apr 22, 2025

The depending core PR has been merged. Would be nice to have a swift review / merge from @openhab/webui-maintainers as the four related PR's cover hundreds of bindings and files and merge conflicts are expected as this is left open for some time.

Copy link
Contributor

@florian-h05 florian-h05 left a comment

Choose a reason for hiding this comment

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

LGTM, thanks!

BTW the WebUI maintainers team is not responsible for the website, only individual persons.
Unfortunately I am not (yet?) one of those.

@lsiepel
Copy link

lsiepel commented Apr 22, 2025

WebUI maintainers team is not responsible for the website, only individual persons.
Unfortunately I am not (yet?) one of those.

Ah, my bad. @kaikreuzer or @stefan-hoehn

@andrewfg
Copy link
Contributor Author

@kaikreuzer or @stefan-hoehn could you please merge this one? (we have a big backlog in addons that depends on this latest version)

@stefan-hoehn stefan-hoehn merged commit 3cf951a into openhab:main Apr 23, 2025
5 checks passed
@andrewfg
Copy link
Contributor Author

Many thanks @stefan-hoehn

@andrewfg andrewfg deleted the update-schema-again branch December 14, 2025 12:10
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