FallingLights / Teachable-dl

Course downloader for teachable platform written in python3 using selenium and yt-dlp
GNU Lesser General Public License v3.0
106 stars 27 forks source link

[BUG] #3

Closed dan1664 closed 1 year ago

dan1664 commented 1 year ago

Describe the bug I followed all the steps and all the modules installed in Ubuntu terminal. After I put in the line below, I saw it open up a Chrome browser, go to the website, login, and then it stayed on the main page which shows all the courses available. Then it shut down Chrome and gave me the below Max retries exceeded error.

(env) danny@danny-VirtualBox:~$ python3 main.py --url https://school.patbailouni.com/courses/challenge21311111 --email dan1664@hotmail.com --password ***** Could not download course: https://school.patbailouni.com/courses/challenge21311111 cause: HTTPConnectionPool(host='localhost', port=43325): Max retries exceeded with url: /session/25c596d8858b4817335e16d7adc01f15/url (Caused by NewConnectionError('<urllib3.connection.HTTPConnection object at 0x7f6eba86a560>: Failed to establish a new connection: [Errno 111] Connection refused'))

Desktop (please complete the following information):

FallingLights commented 1 year ago

@HenestrosaConH @dan1664 Looks like there are more than 2 layouts for the teachable platform ill write the parser for this new one tomorrow

dan1664 commented 1 year ago

@FallingLights amazing, thank you for your time and help!

FallingLights commented 1 year ago

Test the test branch "new-template-test". Did not get a change to test it myself

dan1664 commented 1 year ago

Just tested it and I got this:

env) danny@danny-VirtualBox:~$ python3 main.py --url https://school.patbailouni.com/courses/challenge21311111 --email dan1664@hotmail.com --password ** Could not download course: https://school.patbailouni.com/courses/challenge21311111 cause: HTTPConnectionPool(host='localhost', port=56675): Max retries exceeded with url: /session/cbb4c7c8b53e6733e6d32c790a49e7e7/url (Caused by NewConnectionError('<urllib3.connection.HTTPConnection object at 0x7ff9154b14b0>: Failed to establish a new connection: [Errno 111] Connection refused'))

FallingLights commented 1 year ago

@dan1664 If you could, can you send me your email and password to fallingluminosity@protonmail.com so I can test it locally

dan1664 commented 1 year ago

@FallingLights I just sent it via buymeacoffee, thank you for your time and help on this!

FallingLights commented 1 year ago

@dan1664 Thanks for the coffee! Latest commit on the test branch now works for your site. @HenestrosaConH also give the try to the latest commit

FallingLights commented 1 year ago

@HenestrosaConH Fixed in the latest commit

FallingLights commented 1 year ago

@HenestrosaConH Get latest commit and run with -vv argument and send the output if it still throwing the error

FallingLights commented 1 year ago

@HenestrosaConH Mind elaborating on why you have to complete and continue? So far I have not seen the need to do that for the courses that i have downloaded.

And thanks for coffee!

FallingLights commented 1 year ago

@HenestrosaConH Mind elaborating on why you have to complete and continue? So far I have not seen the need to do that for the courses that i have downloaded. And thanks for coffee!

I tried to download a course that I had not fully completed. When the program tried to download lessons that I had not unlocked, it couldn't find the video because a message appeared saying that I had to complete the previous lesson to unlock the new one.

Here is an example of what I'm saying:

image

Added the option in the commit a94d04d6