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

feat: upgrade Android Auth library to 2.0.2 #210

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

wwdrew
Copy link

@wwdrew wwdrew commented Aug 12, 2022

Fixes #209 .

Changes introduced in this pull request:

  • Update SpotifySDK submodule to latest commit
  • Update build.gradle to import auth-lib from maven
  • Remove clearCookies method call as this no longer exists in new SDK

Tasks

  • Add yourself as contributor via yarn contribute
  • Update Unreleased section of CHANGELOG.md with changes (Convention)

@cjam

Please note I'm having problems testing this change so there's a high chance it doesn't work. However, everything in Android Studio is saying it's okay so I'm hopeful it should work. I just couldn't get a package to build I could import into my project to make sure.

@cjam
Copy link
Owner

cjam commented Aug 22, 2022

Awesome thanks for the PR! sorry for the delay, I've been underwater at work.
I will try and merge and test it as soon as I can.

@wwdrew
Copy link
Author

wwdrew commented Aug 22, 2022

Hi @cjam, no worries I know exactly how it feels :)

I've managed to move this PR forward a little testing-wise and there's still some more work to be done on it, Spotify's changed a few things in the move from v1 -> v2 of the auth sdk that I'm trying to integrate properly.

I've also taken this comment (#203) into consideration and changed how the auth-sdk is imported, as it's the only way to get v2.0.2.

I should probably mark this PR as WIP until I've got it sorted really.

@wwdrew wwdrew changed the title feat: upgrade Android Auth library to 2.0.1 feat: upgrade Android Auth library to 2.0.2 Aug 22, 2022
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.

Android SDK is out-of-date
2 participants