sindresorhus/electron-dl

Use `ses.downloadURL(url)`

Open

#104 ouverte le 5 févr. 2020

Voir sur GitHub
 (2 commentaires) (0 réactions) (0 assignés)JavaScript (1 206 stars) (136 forks)user submission
enhancementhelp wanted

Description

Electron 8 added a new useful API: https://github.com/electron/electron/pull/19889/files

We should use it for electronDl.download().

Guide contributeur

Stack technique
javascriptnodejs
Domaine
desktopapi
Type d'issue
feature
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
2
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
under 1 hour
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
stale
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
clear
Prérequis
Familiarity with Electron's download API
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
70
Direction de recherche
Review the Electron 8 PR #19889 (linked in the issue) to understand the ses.downloadURL API. Then locate the download function in the electronDl module (likely in index.js) and replace the existing download implementation with ses.downloadURL(url). Ensure backward compatibility and update any related tests.