Skip to content
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

switch to axios #4

Open
wants to merge 8 commits into
base: master
Choose a base branch
from
Open

switch to axios #4

wants to merge 8 commits into from

Conversation

Wamy-Dev
Copy link

switch to axios and other general improvements on the main.

@Wamy-Dev
Copy link
Author

Doing this would allow switching from the deprecated "requests" library to the more popular and updated "axios" library on the npm supported repository.

@Wamy-Dev
Copy link
Author

Wamy-Dev commented Aug 16, 2022

allows for easier integration into projects this way while having updated libraries. I doubt this will be merged as this library is extremely old, but @atombarel I hope on your repository, it can be updated with axios...

@atombarel
Copy link

allows for easier integration into projects this way while having updated libraries. I doubt this will be merged as this library is extremely old, but @atombarel I hope on your repository, it can be updated with axios...

New version is out with axios

@Wamy-Dev
Copy link
Author

@atombarel I did some testing, and the axios actually fails when trying to use it. This was my mistake for not even checking, but please check it out. Its not sending links to realdebrid when using /link/unrestrict.

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.

3 participants