Skip to content

Update quick-connect.md #1332

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

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Update quick-connect.md #1332

wants to merge 1 commit into from

Conversation

dbareis
Copy link

@dbareis dbareis commented Mar 11, 2025

Correctly describe the quick connect process as the existing text was not understandable (at least by me).

Correctly describe the quick connect process as the existing text was not understandable (at least by me).
@jellyfin-bot
Copy link

Cloudflare Pages deployment

Latest commit 3e54a27cd9742c71eb5cbf15e0c8b894601433a6
Status ✅ Deployed!
Preview URL https://45c2eba4.jellyfin-org.pages.dev
Type 🔀 Preview

@dbareis
Copy link
Author

dbareis commented Mar 11, 2025

It also wouldn't hurt to get rid of the "got it" button (which you have to press even if you don't got it) and to add some text to the dialog that describes the process and or links to this documentation.

@@ -15,14 +15,17 @@ Settings > Dashboard > General > Enable Quick Connect on this server

## Using Quick Connect

To sign in to a supported client, you have to enter the Quick Connect code in your user settings.
The sign in page of the client has a "**Use Quick Connect**" button which will generate a 6 digit code, take note of the code.
Copy link
Member

@felix920506 felix920506 Apr 27, 2025

Choose a reason for hiding this comment

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

Please mention that not all clients support this. Also, not all clients are like this. For example, on Finamp (3rd party client) it will display the quick connect code by default, and the login with account and password is behind a button.

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