Added example that shows how to sync subscriptions #85
Conversation
|
Thank you for the PR! The changelog has not been updated, so here is a friendly reminder to check if you need to add an entry. |
Codecov Report✅ All modified and coverable lines are covered by tests. 📢 Thoughts on this report? Let us know! |
5c3eece to
e53f161
Compare
|
Thank you for the PR! The changelog has not been updated, so here is a friendly reminder to check if you need to add an entry. |
|
Since our backend changes, this has become much cleaner. Running fine now on my tests. I also started on workflows to run it on github, but the token reneweal and upload logic warrants a 4 eye check (not commiteed yet, cos, well, ugly) |
e53f161 to
068b7de
Compare
068b7de to
d9c84d5
Compare
|
We should clean up the terminology around sync, migrate, and subscribe. Right now, it’s not always clear what we mean when we use these terms. We should formalize the definitions and document them. Here’s a proposal that feels consistent to me:
|
|
Do you think we could make this example more generic? Ideally, given any source playlist, it should be possible to attach multiple subscriber playlists and keep them updated by simply re-triggering the script (or by defining a polling interval). Independent of service specifics. Would be a good intermediate step for us before we start to work on the fully synchronized playlists example. Should also allow us to futher evaluate our abstraction 🙃 |
…e updated regularly on spotify) to tidal
d9c84d5 to
84e723d
Compare
I follow some playlists that are updated regularly on spotify via tidal.
This might be a cool example and good use-case to show.
I was tweaking this a bit more, and after the refactor it works quite well.
If we fix some of the open issues (retry logic) the code should also be much shorter.