Skip to content

Switch index format from HTML to EJS to match upstream changes#14

Merged
thespad merged 1 commit intolinuxserver:mainfrom
meltyshev:main
Mar 20, 2026
Merged

Switch index format from HTML to EJS to match upstream changes#14
thespad merged 1 commit intolinuxserver:mainfrom
meltyshev:main

Conversation

@meltyshev
Copy link
Contributor

linuxserver.io


  • I have read the contributing guideline and understand that I have made the correct modifications

Description:

Update Dockerfiles to build the client with INDEX_FORMAT=ejs and move the resulting index.ejs to /app/views/ instead of using index.html. This matches the upstream PLANKA change and resolves the reported issue (plankanban/planka#1593).

Benefits of this PR and context:

Upstream PLANKA switched the client index format from HTML to EJS (plankanban/planka#1451). Without this change, the app is completely broken because the server tries to load index.ejs instead of index.html, which always results in a 500 error.

How Has This Been Tested?

Verified that the changes in both Dockerfile and Dockerfile.aarch64 are consistent. Built the image and checked that all files are placed in the correct location.

Source / References:

Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

Thanks for opening this pull request! Be sure to follow the pull request template!

@LinuxServer-CI
Copy link
Contributor

I am a bot, here is the pushed image/manifest for this PR:

ghcr.io/linuxserver/lspipepr-planka:v2.1.0-pkg-642512ea-dev-c251caef0a4999d7ec044f1563c9a4dabc8a753a-pr-14

Copy link
Member

@thespad thespad left a comment

Choose a reason for hiding this comment

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

Thanks for this.

@github-project-automation github-project-automation bot moved this from PRs to PRs Approved in Issue & PR Tracker Mar 20, 2026
@thespad thespad merged commit 50157e3 into linuxserver:main Mar 20, 2026
3 checks passed
@LinuxServer-CI LinuxServer-CI moved this from PRs Approved to Done in Issue & PR Tracker Mar 20, 2026
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