Skip to content

UI - #277

Open
Chris230291 wants to merge 3 commits into
linuxserver:masterfrom
Chris230291:master
Open

UI#277
Chris230291 wants to merge 3 commits into
linuxserver:masterfrom
Chris230291:master

Conversation

@Chris230291

Copy link
Copy Markdown
Contributor

Now builds with support for the new UI

@bernhardberger

Copy link
Copy Markdown
Contributor

Thanks for putting this together. I found it while looking into why /gui is disabled in the current image. This looks like the right change.

I checked the repo's contribution guide and noticed a few things that are not covered yet:

  • Link this to [FEAT] Build with support for the new web UI #276 by adding Closes #276 and complete the PR template, including how it was tested
  • Apply the same changes to Dockerfile.aarch64
  • Put npm in the correct alphabetical position in the package list
  • Add a changelog entry to readme-vars.yml

Have you already built it and confirmed that /gui loads? We're using the amd64 image and can test a build if useful.

@Chris230291

Copy link
Copy Markdown
Contributor Author

I built it, I'm using it, it works. Already gone back to the legacy UI if I'm honest.
Can't test arm.
Closes #276

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

Labels

None yet

Projects

Development

Successfully merging this pull request may close these issues.

3 participants