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

Improve and rework MediaService & Android Auto integration #232

Merged
merged 1 commit into from
Nov 15, 2020

Conversation

Maxr1998
Copy link
Member

  • Moves library browser code into extra class and package, as it will only be used for Android Auto.
  • The loader function was split into multiple subfunctions and cleaned up.
  • The format of 'parentIds' was improved, unexpected behaviour with queue selection fixed.
  • Shuffling was temporarily removed and will be handled differently in the future.
  • Also introduces preliminaries for casting and Android 11 seamless transfer, this isn't fully enabled yet however - it seems there are still some changes necessary within the AndroidX libraries.

Uses code from the android/uamp reference media player project: https://github.com/android/uamp

Closes #226.

@Maxr1998 Maxr1998 added bug Something isn't working enhancement New feature or request code-cleanup/refactor Code cleanup or rework/refactor labels Nov 11, 2020
@Maxr1998
Copy link
Member Author

Note: the code for search is untested, because Google Assistant is dumb and doesn't properly recognize queries directed at Jellyfin.
I tested the rest of the app both with the Android Auto phone app and within an actual car, and thankfully didn't observe any problems (apart from search which isn't Jellyfin's fault).

@nielsvanvelzen
Copy link
Member

You can use this app to test the integration - need to build it yourself though.

@Maxr1998
Copy link
Member Author

Maxr1998 commented Nov 12, 2020

Oh wait, I already have that installed! Thanks for reminding me 😅

EDIT: It works great! You unfortunately can't provide those "album" or "artist" extras within the testing app however.

@nielsvanvelzen
Copy link
Member

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

Moves library browser code into extra class and package, as it will only be used for Android Auto.
The loader function was split into multiple subfunctions and cleaned up.
The format of 'parentIds' was improved, unexpected behaviour with queue selection fixed.
Shuffling was temporarily removed and will be handled differently in the future.
Also introduces preliminaries for casting and Android 11 seamless transfer, this isn't fully enabled yet however -
it seems there are still some changes necessary within the AndroidX libraries.

Uses code from the android/uamp reference media player project: https://github.com/android/uamp
@nielsvanvelzen
Copy link
Member

/azp run

we should look into these build issues 😒

@azure-pipelines
Copy link

Command 'run

we' is not supported by Azure Pipelines.



Supported commands

  • help:
    • Get descriptions, examples and documentation about supported commands
    • Example: help "command_name"
  • list:
    • List all pipelines for this repository using a comment.
    • Example: "list"
  • run:
    • Run all pipelines or specific pipelines for this repository using a comment. Use this command by itself to trigger all related pipelines, or specify specific pipelines to run.
    • Example: "run" or "run pipeline_name, pipeline_name, pipeline_name"
  • where:
    • Report back the Azure DevOps orgs that are related to this repository and org
    • Example: "where"

See additional documentation.

@nielsvanvelzen
Copy link
Member

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working code-cleanup/refactor Code cleanup or rework/refactor enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Android Auto Compliance Issues
2 participants