mapbox/mapbox-gl-js

Do not handle video loading errors

Aperta

#5646 aperta il 10 nov 2017

Vedi su GitHubGuida contributor
 (3 commenti) (0 reazioni) (0 assegnatari)JavaScript (2203 fork)batch import
api :memo:good first issue

Metriche repository

Star
 (10.532 stelle)
Metriche merge PR
 (Metriche PR in attesa)

Descrizione

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

Guida contributor

Direzione di ricerca
Esamina la funzione getVideo in src/util/ajax.js, aggiungi la gestione degli errori per il caricamento video e considera di restituire il video tramite callback invece di un ritorno diretto.
Tech stack
javascript
Dominio
frontend
Tipo issue
Bug
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
Meno di 1 ora
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
JavaScriptGit
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
75