mapbox/mapbox-gl-js

Do not handle video loading errors

Open

#5.646 aberto em 10 de nov. de 2017

Ver no GitHub
 (3 comments) (0 reactions) (0 assignees)JavaScript (2.203 forks)batch import
api :memo:good first issue

Métricas do repositório

Stars
 (10.532 stars)
Métricas de merge de PR
 (Nenhuma PRs mesclada em 30d)

Description

In src/util/ajax.js, getVideo seems not handle any loading errors.

And Also, I think it's useless to return video in the end. We should get video in callback.

https://github.com/mapbox/mapbox-gl-js/blob/master/src/util/ajax.js#L149

Guia do colaborador

Direção de pesquisa
Inspecione a função getVideo em src/util/ajax.js, adicione tratamento de erros para carregamento de vídeo e considere retornar o vídeo via callback em vez de retorno direto.
Pilha de tecnologia
javascript
Domain
frontend
Tipo Issue
Bug
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.
Menos de 1 hora
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
Ativo
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Claro
Prerequisites
JavaScriptGit
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
75