Skip to content

fix(settingsplex.tsx): changes the suggested url from plex's "hosted" app to...#3250

Open
ofgrenudo wants to merge 4 commits into
seerr-team:developfrom
ofgrenudo:fix-misleading-plex-web-app-url-issue3220
Open

fix(settingsplex.tsx): changes the suggested url from plex's "hosted" app to...#3250
ofgrenudo wants to merge 4 commits into
seerr-team:developfrom
ofgrenudo:fix-misleading-plex-web-app-url-issue3220

Conversation

@ofgrenudo

@ofgrenudo ofgrenudo commented Jul 15, 2026

Copy link
Copy Markdown
Contributor

changes the suggested url from plex's "hosted" app to recommend the users server url

Description

This changes the suggested url from plex's hosted app I.E. https://app.plex.tv/desktop to recommend setting it to your servers fqdn for example https://your-servers-fqdn.com/web/index.html allowing you to have the users to click on the open in plex button when looking at media and have it open directly on the servers web app.

How Has This Been Tested?

  • have ran pnpm dev
  • have ran pnpm build

Screenshots / Logs (if applicable)

Screenshot From 2026-07-15 12-05-40

Checklist:

  • I have read and followed the contribution guidelines.
  • Disclosed any use of AI (see our policy)
  • I have updated the documentation accordingly.
  • All new and existing tests passed.
  • Successful build pnpm build
  • Translation keys pnpm i18n:extract
  • Database migration (if required)

AI Disclosure

AI was not used in this PR.

Summary by CodeRabbit

Summary by CodeRabbit

  • Documentation
    • Updated Plex web app setup guidance to point users to the default destination URL more explicitly.
    • Changed the web app URL field placeholder to an example server-based path (https://your-server-fqdn.com/web/index.html).

… app to the users server url

This changes the suggested url from plex's hosted app I.E. https://app.plex.tv/desktop to recommend
setting it to your servers fqdn for example https://your-servers-fqdn.com/web/index.html allowing
you to have the users to click on the open in plex button when looking at media and have it open
directly on the servers web app.

fix seerr-team#3220
@ofgrenudo
ofgrenudo requested a review from a team as a code owner July 15, 2026 16:13
@coderabbitai

coderabbitai Bot commented Jul 15, 2026

Copy link
Copy Markdown
Contributor

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: f813815e-f9e2-4886-ab26-76d2cf820647

📥 Commits

Reviewing files that changed from the base of the PR and between 223c030 and b3abed0.

📒 Files selected for processing (1)
  • src/i18n/locale/en.json

📝 Walkthrough

Walkthrough

Plex settings guidance now references https://app.plex.tv/desktop, while the web app URL placeholder demonstrates a server-hosted path.

Changes

Plex URL guidance

Layer / File(s) Summary
Self-hosted URL example
src/components/Settings/SettingsPlex.tsx, src/i18n/locale/en.json
Updates the Plex web app tip and changes the URL placeholder to https://your-server-fqdn.com/web/index.html.

Estimated code review effort: 1 (Trivial) | ~2 minutes

Suggested reviewers: fallenbagel

Poem

A bunny hops through Plex’s page,
With server paths now center stage.
The hosted link is named with care,
A clearer example waits right there.
Hop, hop—URLs bloom!

🚥 Pre-merge checks | ✅ 3 | ❌ 1

❌ Failed checks (1 inconclusive)

Check name Status Explanation Resolution
Title check ❓ Inconclusive The title is related to the Plex URL change but is incomplete and ends with an ellipsis. Use a complete title like "fix(settingsplex.tsx): update Plex Web App URL suggestion for self-hosted servers".
✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Linked Issues check ✅ Passed The PR updates the Plex Web App URL guidance and placeholder to a self-hosted format as requested in #3220.
Out of Scope Changes check ✅ Passed The changes stay within the Plex settings documentation and translation update scope, with no obvious unrelated edits.

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@0xSysR3ll

0xSysR3ll commented Jul 16, 2026

Copy link
Copy Markdown
Contributor

Did you run `pnpm i18n:extract" to update the translations ?

i ran this earlier but didnt work
@ofgrenudo

Copy link
Copy Markdown
Contributor Author

Did you run `pnpm i18n:extract" to update the translations ?

i did, but i guess i made a change after i ran it and didnt run it again. It should be good now.

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.

[Docs]: Plex Web App URL

2 participants