[Release-1.0] Apple Music: When adding a track on Grüvee we should add the track to the platform playlist - Githubissues

PixelogicDev / Gruvee-Mobile

#WeAreGrüvee

MIT License

48 stars 11 forks source link

[Release-1.0] Apple Music: When adding a track on Grüvee we should add the track to the platform playlist #97

Closed adilanchian closed 3 years ago

adilanchian commented 4 years ago

This task combines a couple things here:

[x] Add support for adding Apple Music media to Grüvee

General

This flow for this should be:

Add Apple Music song link to app
Verify proper url link
Get metadata for song
Check to see if song already exists in Grüvee database
Return song data to user & add to playlist document

adilanchian commented 4 years ago

What We Did Today

Planned and wrote down some thoughts on how to handle this scenario
Tweaked some document data to adhere to new structure
Wrote the logic for checking collection if the song already exists

What We Will Do Next Time

[x] Write logic for calling Apple endpoint to get data for song, playlist, album

adilanchian commented 4 years ago

What We Did Today

Wrote some more logic for Apple Music Endpoints
Was able to extract more functions into another shared packaged
Was able to call Get Track endpoint from Apple Music successfully!

What We Will Do Next Time

[x] Update client regex to get storefront (probably look into how this is handled in other countries)
[x] Continue creating song object to store in Firebase

adilanchian commented 4 years ago

What We Did Today

Finished up the regex for storefront
Tweaked all the song objects in the client and database to reflect new schema
Started working on calling Get Playlist API for Apple Music

What We Will Do Next Time

[x] Finish up the Get Playlist API call for Apple Music
[ ] Finish up The Get Album API call for Apple Music

adilanchian commented 3 years ago

What We Did Today

Today we finished up the Playlist API call for Apple Music
Added a bunch of new Apple Music types (SWEET)

What We Will Do Next Time

[x] Finish up The Get Album API call for Apple Music
[ ] Finalize Firebase rules for Songs/Comments
[ ] Start planning the fetching of other services when adding a new song

adilanchian commented 3 years ago

What We Did Today

Finished implementing the Get Album API work for Apple Music
Started cleaning up Spotify Get Media function (still have a bit more to go here)
Fixed some stuff in the client app to show proper Spotify song album artwork

What We Do Next Time

[x] Finish cleaning up Spotify function. We need to start using a non-specific user token (dev token only)
[ ] Finalize Firebase rules for Songs/Comments
[ ] Start planning the fetching of other services when adding a new song

adilanchian commented 3 years ago

What We Did Today

Today we spent the stream cleaning up all the GetSpotifyMedia Firebase function
This included adding a proper check for our AuthToken in Firebase

What We Will Do Next Time

[x] Finalize Firebase rules for Songs/Comments
[x] Start planning/implementing the fetching of other services when adding a new song

adilanchian commented 3 years ago

What We Did Today

Today we worked on fixing some Firestore Function rules (turned out it was just some weird code on the client)
Cleaned up the weird on the client
Started planning and implementing our new trigger function for fetching media for all providers when a new song document is created!

What We Will Do Next Tme

[x] Continue setting up trigger function
[ ] Clean up the structs to include a trigger event value solely for users (follow songs example)

adilanchian commented 3 years ago

What We Did Today

Ran into an Apple Authentication issue (Will be investigating offline)
Continued working on our FetchAllMedia function
This is almost ready to test, we are just cleaning up types to be used throughout other functions and for our new search endpoints

What We Will Do Next Time

[x] Continue cleaning structs
[x] Finalize the Spotify query search endpoint and test!

adilanchian commented 3 years ago

What We Did Today

Continued working on getting the trigger function setup. We got the query to work great
We then needed to test the actual trigger, and unfortunately, that only works in the cloud atm. BUT we were able to test the Github actions and get it working!
Just ran into a small snag with tagging the pkg folder versions

What We Will Do Next Time

[ ] Finish verifying Github Actions
[ ] Test song query trigger function on mobile
[ ] Move on to adding query support for Apple Music

adilanchian commented 3 years ago

We took a bit of a turn today and decided to work on getting the gruveebackend repo functions setup with proper logging and adding documentation around it. Will continue to work on this and then deploy all the functions to the cloud using our fancy Github Actions.

adilanchian commented 3 years ago

A LOT has happened in the past few days:

Updated ALL the logging in the our Firebase functions to get a better understanding of what's going on in production
Got the hang of how we are utilizing the Github actions for proper cloud deployment
Tested and deployed FetchAllMedia trigger function that is currently working if you add a song in Apple Music, it will get the proper data in Spotify.

While now this is a tad off topic for this issue, it was highly needed. the last task for this issue is to get the "Add to playlist" support going

adilanchian commented 3 years ago

One thing we have to think about here is when we add things such as albums and playlists, we need to traverse through all of the tracks and add them separately to the playlist... This is not a current issue we are currently ready for. I think we should make a separate issue for this and work on that as part of the Alpha v3 launch. It's not as highly important as using the actual app