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

Adapt to core changes (addon.xml) #1468

Merged
merged 8 commits into from
Jan 15, 2023
Merged

Conversation

J-N-K
Copy link
Member

@J-N-K J-N-K commented Aug 18, 2022

Depends on #1452
Depends on openhab/openhab-core#3050

@J-N-K
Copy link
Member Author

J-N-K commented Aug 18, 2022

I didn't see an add-on for the iconset. Can it be installed separately?

@wborn
Copy link
Member

wborn commented Aug 19, 2022

The iconset is always installed to prevent issues, see openhab/openhab-distro#1371.

@kaikreuzer
Copy link
Member

@J-N-K Is there a special reason, why this PR is still marked as "WIP" in the title?
As a heads-up: I'd plan to finish my review on openhab/openhab-core#3050 tomorrow and would then like to merge it - in consequence, we will have to merge the PRs in the other repos as well.

@J-N-K
Copy link
Member Author

J-N-K commented Jan 14, 2023

To be honest: I can't remember. I'll check tomorrow. After the core changes are approved I need a day or two to finalize the addons-PR since new addons were added since I created it. I just want to prevent double-work there in case something needs to be changed.

@J-N-K J-N-K changed the title [WIP] Adapt to core changes (addon.xml) Adapt to core changes (addon.xml) Jan 15, 2023
@J-N-K
Copy link
Member Author

J-N-K commented Jan 15, 2023

I fixed some minor issues, ready now from my side

@relativeci
Copy link

relativeci bot commented Jan 15, 2023

Job #697: Bundle Size — 15.97MiB (~+0.01%).

eb3190e(current) vs 9471df4 main#696(baseline)

Metrics (1 change)
                 Current
Job #697
     Baseline
Job #696
Initial JS 1.73MiB(~-0.01%) 1.73MiB
Initial CSS 608.52KiB 608.52KiB
Cache Invalidation 90.48% 90.48%
Chunks 218 218
Assets 688 688
Modules 2011 2011
Duplicate Modules 108 108
Duplicate Code 1.8% 1.8%
Packages 133 133
Duplicate Packages 15 15
Total size by type (2 changes)
                 Current
Job #697
     Baseline
Job #696
CSS 856.56KiB 856.56KiB
Fonts 1.08MiB 1.08MiB
HTML 1.23KiB 1.23KiB
IMG 140.74KiB 140.74KiB
JS 9.04MiB (~-0.01%) 9.04MiB
Media 295.6KiB 295.6KiB
Other 4.59MiB (~+0.01%) 4.59MiB

View job #697 reportView main branch activity

@kaikreuzer
Copy link
Member

Is the file binding-config.vue still required?

@J-N-K
Copy link
Member Author

J-N-K commented Jan 15, 2023

No, I removed it.

J-N-K added 8 commits January 15, 2023 20:32
Signed-off-by: Jan N. Klug <[email protected]>
This reverts commit f0ab4e42578131cffaed6f48941083a3b7891eaa.

Signed-off-by: Jan N. Klug <[email protected]>
Signed-off-by: Jan N. Klug <[email protected]>
Signed-off-by: Jan N. Klug <[email protected]>
Signed-off-by: Jan N. Klug <[email protected]>
Signed-off-by: Jan N. Klug <[email protected]>
@kaikreuzer kaikreuzer merged commit e012d94 into openhab:main Jan 15, 2023
crnjan pushed a commit to crnjan/openhab-webui that referenced this pull request Jan 17, 2023
Signed-off-by: Jan N. Klug <[email protected]>
Signed-off-by: Boris Krivonog <[email protected]>
@J-N-K J-N-K deleted the feature-addon branch January 17, 2023 17:16
wborn added a commit to wborn/openhab-webui that referenced this pull request Aug 21, 2023
It is still using /bindings to get the data resulting in a 404 Not Found.
The other pages were updated in openhab#1468.

Signed-off-by: Wouter Born <[email protected]>
florian-h05 pushed a commit that referenced this pull request Aug 29, 2023
It is still using /bindings to get the data resulting in a 404 Not
Found. The other pages were updated in #1468.

Signed-off-by: Wouter Born <[email protected]>
florian-h05 added a commit that referenced this pull request Aug 30, 2023
It is still using /bindings to get the data resulting in a 404 Not
Found. The other pages were updated in #1468.

Signed-off-by: Wouter Born <[email protected]>
(cherry-picked from commit 312aa0e)
stefan-hoehn pushed a commit to stefan-hoehn/openhab-webui that referenced this pull request Sep 23, 2023
It is still using /bindings to get the data resulting in a 404 Not
Found. The other pages were updated in openhab#1468.

Signed-off-by: Wouter Born <[email protected]>
Signed-off-by: Stefan Höhn <[email protected]>
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.

3 participants