Skip to content

Conversation

@CosmicSage
Copy link

I successfully add listening history download to scdl. In many of my attempts to download I have always got the desired result .

My Code is very non-invasive to the codebase almost like blue in sky, as you might expect.

Ok so the test may fail because @7x11x13 of soundcloud.py API wrapper hasn't yet merged my work of adding history endpoint to the wrapper whose PR you can see here

So to test on your computer you can check my fork and copy paste two files to your site-packages (that's what I did in development)

Look if this gets merged I will make it cleaner and enhance this new feature as well, as the api returns few more data fields which are not considered while downloading.

Hope you like it. Please merge fast

Thank you

@CosmicSage
Copy link
Author

Here's the screenshot of results
Screenshot (470)

You must copy the main soundcloud.py and resource/history.py from soundcloud api wrapper fork to site-packages to test this locally as @7x11x13 hasn't merged/released my code yet/

@7x11x13 @flyingrub

@CosmicSage
Copy link
Author

Also can I get mordertor access I will take care I swear..🙏 @7x11x13 @flyingrub

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.

1 participant