-
Notifications
You must be signed in to change notification settings - Fork 81
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
Auth for android #99
Comments
@Nirmitjatana it seems that you didn't add all modules, you have to add auth and appremote. Follow the instructions from the readme, especially this part from https://developer.spotify.com/documentation/android/quick-start/ |
Ok so this is an issue that we have to fix Multiple things I found out: The newest release of android studio removed the option to add .jar/.aar files the old way via the dialogue. Here it states the the Project Structure Dialogue is the new way to go: https://developer.android.com/studio/projects/android-library#psd-add-library-dependency And this states on how to add a .jar/.aar as a dependecy: https://developer.android.com/studio/projects/android-library#psd-add-aar-jar-dependency The issue we encounter then is that our linking will probably not work but I am not sure yet. We have to do the linking currently because spotify did not put their sdk into a maven repository, what would be the recommended way of doing things
I found out that the auth sdk is already in a maven repo, so we can change that, so that you as a user don't have to add this anymore. -> spotify/android-sdk#271 But the app remote currently has to be added the old/current way. The optimal way would be that a user does not have to do anything in order to use the package, and I'm investigating if this is doable. |
@brim-borium Thanks for the detailed explanation, will look into it. |
When I tried with the old version of Android Studio where it have option to add .jar/.aar files via the dialogue it worked successfully. |
I've faced the same problem on Android Studio 4.2. Is there any solution to this problem? |
Thanks for the solution, do you think that we could also embed this into the package so nobody has to do this manual work? |
@brim-borium |
@brim-borium reminder to add this solution to the documentation |
I am getting this issue, while trying to run the app. I am only using auth service as of now
Screenshots
![image](https://user-images.githubusercontent.com/52108077/117544269-45c60b80-b03e-11eb-8b4d-dfef10babbfd.png)
The text was updated successfully, but these errors were encountered: