-
Notifications
You must be signed in to change notification settings - Fork 299
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
Fix failing commands by upgrading to latest Udacity API #230
base: master
Are you sure you want to change the base?
Conversation
If I may ask, how can i compile & install your branch? Unfortunately, couldn't find a build file. These are my trials: Output: (Failure)
Node & npm versions:
|
@OmarShawky1 did you manage to get it running? |
No |
Sorry for late the reply @OmarShawky1. This is how I used it. Remove previous global installation of udacimak npm uninstall -g udacimak Clone the repo https://github.com/Alameen688/fix-udacimak git clone https://github.com/Alameen688/fix-udacimak In the terminal, navigate to the fix-udacimak project cd fix-udacimak Install dependencies npm install Build the project npm run build Install the cli globally npm i -g . Use udacimak udacimak settoken YOUR_BEARER_TOKEN You can test that things are okay by listing your nanodegree courses udacimak listnd You should be able to download courses now. |
@Alameen688 awesome! Just tested it and it works fine. Thank you! |
Great @ChristophBurger89. Thanks for letting me know. |
Sorry for the unintentional late reply, I don't have access to a device in the meantime but I shall be soon and I'll test the pull request and let you know. Thanks in advance for the effort. |
Hello Again, I installed the program with no issues; However, I wasn't able to set the token successful; This is the result of
PS: I used this google chrome extension to grab the cookies: |
@ChristophBurger89 Where you able to authenticate? |
@OmarShawky1 you would need to open the then use the value in the command line
|
Hello Back @Alameen688 ; Again sorry for the delayed reply. I have tested it and it is working perfectly. Thanks to your incredible help, support and patience. Please, Have a nice day :) |
No worries at all @OmarShawky1, we all have other things to do. Happy to know it works. Thanks for letting me know. |
A tip for the future people; Use yt-dlp instead of youtube-dl as it is quite faster (at least 100x)
|
Thank you! Your fix (https://github.com/Alameen688/fix-udacimak) totally worked for me! |
@Alameen688 Bro can you please help changing the video resolution option i tried downloadYoutube.js & other methods didn't work it downloading with quality 22 which is 720 but want to download 137+140 format which will be 1080p video + good audio m4a merge how can i implement this to work properly? can you help me in this :-( More info >> #122 (comment) |
Description
udacimak CLI commands return a connection error during usage
Fixes # (issue)
In the current version, try to authenticate using:
OR
You should get an error similar to:
See full details on how to use this fix - #230 (comment)
Type of change
How Has This Been Tested?
Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration
Configurations
Checklist: