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

Update reverse-proxy.mdx #1052

Open
wants to merge 2 commits into
base: develop
Choose a base branch
from
Open

Update reverse-proxy.mdx #1052

wants to merge 2 commits into from

Conversation

C4J3
Copy link

@C4J3 C4J3 commented Oct 23, 2024

Fix posters not showing up when using sub-folder (at least with NPM)

Description

Fixes title card images not appearing if using subfolders.

Screenshot (if UI-related)

Before:
image

After:
image

Aditional Notes

I know this is not officially supported but I figured this was a byproduct of the recent update and it just wasn't thought about. Small change but for idiots like me it'll save 10-100~ mins of trying to troubleshoot 😅.

Fix posters not showing up when using sub-folder (at least with NPM)
Added `avatarproxy` support

Co-authored-by: Gauthier <[email protected]>
@C4J3
Copy link
Author

C4J3 commented Oct 23, 2024

Possibly unrelated to this. Is the subpath not meant to be maintained for actual jellyseerr subpaths? So if I have mydomain.tld/jellyseer I go to the discovery page but if I click movies I go to mydomain.tld/discover/movies. Don't remeber if I had this behaviour previously because I mostly used requestrr and didn't use external address when connecting to Jellyseerr itself.

@Fallenbagel
Copy link
Owner

Possibly unrelated to this. Is the subpath not meant to be maintained for actual jellyseerr subpaths? So if I have mydomain.tld/jellyseer I go to the discovery page but if I click movies I go to mydomain.tld/discover/movies. Don't remeber if I had this behaviour previously because I mostly used requestrr and didn't use external address when connecting to Jellyseerr itself.

Subpath is not supported officially.

@Fallenbagel
Copy link
Owner

Also the commit message has to start from docs: ..... or this won't be reflected on the docs after merge

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