search

SwapnilSoni1999 / learningcrux-dl

Downloader for learningcrux [BETA]
The Unlicense
11 stars 7 forks source link

Require some changes in modules #7

Open msdsunny opened 3 years ago

msdsunny commented 3 years ago

Hello developers,

I have gone through all of your code but their are some minor and major issues i have found

Major Issue

This issue is about library have used in this project like youtube-dl which has been taken down now so we have to move to another downloader.

Minor Issue

When we run the program the header request is sending request on this https://learningcrux.com/play/course_url/0/0 url so you have to do some change here instead of play you should replace video over there in the url. I have done this patch by merging but still it is not reflecting.

SwapnilSoni1999 commented 3 years ago

youtube-dl is back and I will see about the minor issue in future soon