sindresorhus/electron-dl

Use `ses.downloadURL(url)`

Open

#104 aberto em 5 de fev. de 2020

Ver no GitHub
 (2 comments) (0 reactions) (0 assignees)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().

Guia do colaborador

Pilha de tecnologia
javascriptnodejs
Domain
desktopapi
Tipo Issue
feature
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
2
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
under 1 hour
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
stale
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
clear
Prerequisites
Familiarity with Electron's download API
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
70
Direção de pesquisa
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.