-
-
Notifications
You must be signed in to change notification settings - Fork 219
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 support for external player choice inside client settings #148
Conversation
7532fee
to
6f26e4d
Compare
3d73217
to
e6685e5
Compare
Since we're targeting SDK 30 we need to add the apps we query to the manifest: https://developer.android.com/about/versions/11/privacy/package-visibility |
e6685e5
to
407cec9
Compare
Ok, new changes:
Note: New warning in Android Studio 4.1.
|
Android 10+ makes use of the new Storage API IIRC, there still is a way to get the old behavior though (but for how long 🤔) |
You can safely ignore it, we only use |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Code review only, did not test it yet.
app/src/main/java/org/jellyfin/mobile/settings/SettingsFragment.kt
Outdated
Show resolved
Hide resolved
407cec9
to
64c2351
Compare
app/src/main/java/org/jellyfin/mobile/settings/ExternalPlayerType.java
Outdated
Show resolved
Hide resolved
app/src/main/java/org/jellyfin/mobile/settings/SettingsFragment.kt
Outdated
Show resolved
Hide resolved
app/src/main/java/org/jellyfin/mobile/settings/ExternalPlayerType.java
Outdated
Show resolved
Hide resolved
64c2351
to
1a379e9
Compare
Co-authored-by: Niels van Velzen <git@ndat.nl> Co-authored-by: Max Rumpf <max.rumpf1998@gmail.com>
1a379e9
to
bac9f01
Compare
Thanks for the changes, I'll give it another thorough review tomorrow 👍 |
System choice by default.
Useful to force playback on a specific player without depending on android settings.
For now it only shows a list of the players that support playback tracking as long as they are installed and enabled.