Skip to content

Conversation

digitaldan
Copy link
Contributor

This is in preparation for the upcoming Matter binding. I'm posting a bit early for visibility, still need to test this quite a bit.

See openhab/openhab-addons#18486

Signed-off-by: Dan Cunningham <dan@digitaldan.com>
@digitaldan digitaldan added the enhancement New feature or request label Apr 5, 2025
@digitaldan digitaldan added this to the 5.0 milestone Apr 5, 2025
@florian-h05 florian-h05 added the main ui Main UI label Apr 5, 2025
Signed-off-by: Dan Cunningham <dan@digitaldan.com>
@relativeci
Copy link

relativeci bot commented Apr 5, 2025

#3171 Bundle Size — 11.26MiB (+0.13%).

20d839f(current) vs 60d7f8c main#3157(baseline)

Warning

Bundle contains 2 duplicate packages – View duplicate packages

Bundle metrics  Change 2 changes
                 Current
#3171
     Baseline
#3157
No change  Initial JS 2.01MiB 2.01MiB
No change  Initial CSS 577.53KiB 577.53KiB
Change  Cache Invalidation 18.8% 18.71%
No change  Chunks 251 251
No change  Assets 274 274
Change  Modules 2992(+0.13%) 2988
No change  Duplicate Modules 148 148
No change  Duplicate Code 1.69% 1.69%
No change  Packages 97 97
No change  Duplicate Packages 2 2
Bundle size by type  Change 1 change Regression 1 regression
                 Current
#3171
     Baseline
#3157
Regression  JS 9.47MiB (+0.15%) 9.46MiB
No change  CSS 870.07KiB 870.07KiB
No change  Fonts 526.1KiB 526.1KiB
No change  Media 295.6KiB 295.6KiB
No change  IMG 140.74KiB 140.74KiB
No change  HTML 1.38KiB 1.38KiB
No change  Other 871B 871B

Bundle analysis reportBranch digitaldan:matter-metadataProject dashboard


Generated by RelativeCIDocumentationReport issue

Signed-off-by: Dan Cunningham <dan@digitaldan.com>
Signed-off-by: Dan Cunningham <dan@digitaldan.com>
Signed-off-by: Dan Cunningham <dan@digitaldan.com>
Signed-off-by: Dan Cunningham <dan@digitaldan.com>
@digitaldan digitaldan changed the title [WIP] Matter metadata Matter metadata May 26, 2025
@digitaldan
Copy link
Contributor Author

Hi, @florian-h05 this is ready for review when you get the chance, thanks!

@florian-h05
Copy link
Contributor

I’ve just started reviewing the binding PR, hope I can finish it today (had to leave my keyboard for now).

@digitaldan
Copy link
Contributor Author

Thanks! No worries if not today ;-)

digitaldan and others added 8 commits May 26, 2025 14:27
Signed-off-by: Dan Cunningham <dan@digitaldan.com>
Signed-off-by: Dan Cunningham <dan@digitaldan.com>
Signed-off-by: Florian Hotze <dev@florianhotze.com>
Signed-off-by: Florian Hotze <dev@florianhotze.com>
Signed-off-by: Florian Hotze <dev@florianhotze.com>
Signed-off-by: Florian Hotze <dev@florianhotze.com>
…imal values

Stepsize is 0.01.

Signed-off-by: Florian Hotze <dev@florianhotze.com>
Signed-off-by: Florian Hotze <dev@florianhotze.com>
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!
I have fixed/improved a few things that I noticed, please have a look at my comments and confirm they are correct.

@florian-h05 florian-h05 changed the title Matter metadata Add Matter metadata May 28, 2025
@digitaldan
Copy link
Contributor Author

Thanks @florian-h05 for the additions and review! LGTM

@florian-h05 florian-h05 merged commit 6c66733 into openhab:main May 28, 2025
5 checks passed
@ghys
Copy link
Member

ghys commented May 29, 2025

Congrats all around! 💯

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request main ui Main UI

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants