Skip to content

Copy the original transmission UI in the transmission-web-control mod#974

Closed
edvincent wants to merge 1 commit intolinuxserver:transmission-transmission-web-controlfrom
edvincent:transmission-transmission-web-control-with-original
Closed

Copy the original transmission UI in the transmission-web-control mod#974
edvincent wants to merge 1 commit intolinuxserver:transmission-transmission-web-controlfrom
edvincent:transmission-transmission-web-control-with-original

Conversation

@edvincent
Copy link

linuxserver.io


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

Description:

The transmission-web-control includes a link on the top-right to open the original Transmission UI, which is broken when using this mod. This is because when installing the theme using the install script provided by the theme developers, it renames the original UI as index.original.html - and keeps the other assets with the same name as they don't clash.

Benefits of this PR and context:

Making the existing UI fully working, which involves being able to open the original UI under index.original.html as that's what the theme has a link to.

How Has This Been Tested?

Created my fork repo and dockerhub, built the mod with the github actions and deployed locally. Confirmed the link now works.

https://github.com/edvincent/docker-mods/tree/transmission-transmission-web-control

https://hub.docker.com/layers/edouardv/mods/transmission-transmission-web-control/images/sha256-5c5ef6ac19fe6479928df777c571e5070d26471599e3c299ae90b097e21f2cc6?context=explore

Source / References:

@LinuxServer-CI
Copy link

PR build pushed to ghcr.io/linuxserver/mods:pull_request_974

@LinuxServer-CI
Copy link

This pull request has been automatically marked as stale because it has not had recent activity. This might be due to missing feedback from OP. It will be closed if no further activity occurs. Thank you for your contributions.

@edvincent
Copy link
Author

Hey team, what needs to happen here?

@LinuxServer-CI
Copy link

This pull request has been automatically marked as stale because it has not had recent activity. This might be due to missing feedback from OP. It will be closed if no further activity occurs. Thank you for your contributions.

Comment on lines +6 to +8
mv /usr/share/transmission/public_html/index.html /usr/share/transmission/public_html/index.original.html
cp -R /usr/share/transmission/public_html/* /themes/transmission-web-control/
mv /usr/share/transmission/public_html/index.original.html /usr/share/transmission/public_html/index.html
Copy link
Member

Choose a reason for hiding this comment

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

To minimise the risk of breakage now and in future it might be preferable to do this as:

cp /usr/share/transmission/public_html/index.html /themes/transmission-web-control/index.original.html
cp -nR /usr/share/transmission/public_html/* /themes/transmission-web-control/

@LinuxServer-CI
Copy link

This pull request has been automatically marked as stale because it has not had recent activity. This might be due to missing feedback from OP. It will be closed if no further activity occurs. Thank you for your contributions.

@LinuxServer-CI
Copy link

This pull request is locked due to inactivity

@linuxserver linuxserver locked as resolved and limited conversation to collaborators May 9, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

3 participants