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

[stable26] Replace deprecated "request" with "axios" #741

Closed
wants to merge 2 commits into from

Conversation

backportbot-nextcloud[bot]
Copy link

@backportbot-nextcloud backportbot-nextcloud bot commented Apr 29, 2023

  • ⚠️ This backport had conflicts that were resolved with the 'ours' merge strategy and is likely incomplete

backport of #740

"request" was used only to get the "pdfjs-XXX-dist.zip" file when
building the app, but it was not used in the app itself.

It seems that it is not possible (with axios options only) to throttle
the "onDownloadProgress" event. Nevertheless, an external throttle
function was not used, as the event is not triggered very often and
therefore it does not need to be throttled.

Signed-off-by: Daniel Calviño Sánchez <[email protected]>
@backportbot-nextcloud backportbot-nextcloud bot added bug dependencies Pull requests that update a dependency file labels Apr 29, 2023
@backportbot-nextcloud backportbot-nextcloud bot added this to the Nextcloud 26.0.2 milestone Apr 29, 2023
@szaimen szaimen closed this Apr 29, 2023
@szaimen szaimen deleted the backport/740/stable26 branch April 29, 2023 10:19
@danxuliu
Copy link
Member

danxuliu commented Aug 8, 2023

Fixed and reopened in #793

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug dependencies Pull requests that update a dependency file
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants