cuappdev / tempo

An open-source iOS app for music discovery and sharing with friends.

MIT License

15 stars 1 forks source link

Track saving should support multiple accounts, updates saved tracks o… #253

Closed ChenJesse closed 7 years ago

ChenJesse commented 7 years ago

…n logging in and out of Spotify

190

Use username for the UserDefaults key, in case one phone is used for multiple Spotify accounts
Fix bug where on logging into Spotify, the savedTracks aren't being loaded from the UserDefaults
Fix bug where on logging in/out of Spotify, the playerCells add buttons are not updated immediately

keivanshahida commented 7 years ago

Looks good to me, I just like to be careful with force unwrapping, so double checking where you force unwrap is my only comment.

ChenJesse commented 7 years ago

Yeah I only force unwrap currentSpotifyUser on the right side because it's not force unwrapped on the left side, so it should never get to the right side if the value is nil