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

Add examples to LabelPublishedServerUriHelp string #6163

Merged

Conversation

scottmckenzie
Copy link
Contributor

Adds some examples to the English help text for the Published Server URIs setting.

@scottmckenzie scottmckenzie requested a review from a team as a code owner October 4, 2024 09:17
Copy link
Member

@thornbill thornbill left a comment

Choose a reason for hiding this comment

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

Changes should only be made to the en-us localization file. All other translations are managed in our weblate instance.

@thornbill thornbill added enhancement Improve existing functionality or small fixes ui & ux This PR or issue mainly concerns UI & UX labels Oct 8, 2024
@@ -837,7 +837,7 @@
"LabelPublicHttpsPort": "Public HTTPS port number",
"LabelPublicHttpsPortHelp": "The public port number that should be mapped to the local HTTPS port.",
"LabelPublishedServerUri": "Published Server URIs",
"LabelPublishedServerUriHelp": "Override the URI used by Jellyfin, based on the interface, or client IP address.",
"LabelPublishedServerUriHelp": "Override the URI used by Jellyfin, based on the interface, or client IP address. For example: internal=http://jellyin.example.com, or external=https://jellyfin.example.com, or all=https://jellyfin.example.com",
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
"LabelPublishedServerUriHelp": "Override the URI used by Jellyfin, based on the interface, or client IP address. For example: internal=http://jellyin.example.com, or external=https://jellyfin.example.com, or all=https://jellyfin.example.com",
"LabelPublishedServerUriHelp": "Override the URI used by Jellyfin, based on the interface, or client IP address. For example: internal=http://jellyin.example.com, external=https://jellyfin.example.com, or all=https://jellyfin.example.com",

@scottmckenzie scottmckenzie force-pushed the add-published-server-uri-examples branch from 101f825 to b94c037 Compare October 10, 2024 09:09
Copy link

@jellyfin-bot
Copy link
Collaborator

Cloudflare Pages deployment

Latest commit 583ead304ad6ddcb0650effa179d97948644107c
Status ✅ Deployed!
Preview URL https://3bd9b56e.jellyfin-web.pages.dev
Type 🔀 Preview

@thornbill thornbill enabled auto-merge (squash) October 10, 2024 20:39
@thornbill thornbill added this to the v10.10.0 milestone Oct 10, 2024
@thornbill thornbill merged commit 9536a37 into jellyfin:master Oct 10, 2024
12 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement Improve existing functionality or small fixes ui & ux This PR or issue mainly concerns UI & UX
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

3 participants