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

Update Google music models and Import/Export adapters #1259

Merged
merged 5 commits into from Jul 26, 2023

Conversation

gauss0818
Copy link
Collaborator

  • With the latest change in Youtube Music Library Public API, we need to make the same change to google music models.
  • We will not the insertion token for playlist anymore. Remove cache mapping in import adapter.
  • Add music to demo server.

@gauss0818 gauss0818 requested a review from jzacsh July 14, 2023 17:55
jzacsh
jzacsh previously approved these changes Jul 19, 2023
@gauss0818 gauss0818 force-pushed the google-music-import-export branch 2 times, most recently from 1ff2de1 to 006d966 Compare July 26, 2023 16:30
@jzacsh jzacsh merged commit 97f6260 into master Jul 26, 2023
5 checks passed
@jzacsh jzacsh deleted the google-music-import-export branch July 26, 2023 16:58
alexeyqu-fb pushed a commit to alexeyqu-fb/data-transfer-project that referenced this pull request Aug 15, 2023
Summary:
* Update Google music models and  Import/Export adapters with latest change in music public API

* Add MUSIC service to GoogleTransferExtension; Add test usage code to GoogleMusicExporter

* Add TODO to remove testing usage code

* Add comment for null parameter

Test Plan: 

Reviewers: 

Subscribers: 

Tasks: 

Tags: 


Differential Revision: https://phabricator.intern.facebook.com/D48350791
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.

None yet

2 participants