-
Notifications
You must be signed in to change notification settings - Fork 56
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
Introduce V3.0 #39
Introduce V3.0 #39
Conversation
Hey, The refactored login with selenium now works for me, I tried it using set-token:
Then courses seems to work out too, although there are many completed courses that Datacamp no longer provides:
I was able to download course 1:
However, other courses failed, some worse than others...
|
Thank you for providing great functional test. You should use the id of the course(s) with the download command instead of the sequential numbering. Please try the download command again with the course ID and let me know the result. I will look into showing a warning for that. If you have already completed the old courses and been able to download them, I think this should be fine. Otherwise, I have to see where the problem lies. |
Hi, Ok, I should have been more careful and mindfull of your documentation when doing my test.
I have found that course videos, exercises, markdown files, datasets are all downloaded successfully. Additionally I tested the tracks command, it seems to work too for me.
|
No worries, that's a problem on my end, there is a bug that will be fixed soon. I will also default the sequential numbering, which is more convenient in my opinion, in the download command and will remove the ID column. Once again thank you for the tests. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM overall, it's a great tool. One thing, I encountered this error when trying to login for the first time.
ERROR: Message: stale element reference: element is not attached to the page document
Changes
This PR introduces major changes to the library. These changes are summarized as follows:
requests
is still used to download materials (slides,videos,...).black
andisort
I guess I didn't miss any big change. I would like to hear comments about this before merging. Any feedback would be greatly appreciated.
Test
To test the changes, it is recommended to create a new virtual environment first, then clone the
v3.0
repo with:Now run
datacamp
as per the README file.TODO before merging