-
-
Notifications
You must be signed in to change notification settings - Fork 3.6k
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
removed UIs as they are now at openhab-webui #4923
Conversation
Signed-off-by: Kai Kreuzer <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks Kai!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can you also remove all /addons/ui
dirs from the CODEOWNERS file?
I'll change the codeowners in a follow up |
We still have a couple of UI issues in the issue tracker. Can you use your powers to transfer these to the webui repo @kaikreuzer ? These are the UI issues I found:
|
@wborn Done! |
Signed-off-by: Kai Kreuzer <[email protected]> Signed-off-by: Pshatsillo <[email protected]>
Signed-off-by: Kai Kreuzer <[email protected]> Signed-off-by: Maximilian Hess <[email protected]>
UIs had only been moved temporarily here, but are now located at https://github.com/openhab/openhab-webui.
This should speed up the addons build again and especially solve the OOM issues that npm causes, so that we can reactivate SAT check and unit test execution.
Signed-off-by: Kai Kreuzer [email protected]