mapbox/mapbox-gl-js

Do not handle video loading errors

Open

#5.646 geöffnet am 10. Nov. 2017

Auf GitHub ansehen
 (3 Kommentare) (0 Reaktionen) (0 zugewiesene Personen)JavaScript (2.203 Forks)batch import
api :memo:good first issue

Repository-Metriken

Stars
 (10.532 Stars)
PR-Merge-Metriken
 (Keine gemergten PRs in 30 T)

Beschreibung

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

Contributor Guide

Research-Richtung
Untersuchen Sie die getVideo Funktion in src/util/ajax.js, fügen Sie eine Fehlerbehandlung für das Laden von Videos hinzu und erwägen Sie, das Video über einen Callback zurückzugeben anstatt direkt.
Tech Stack
javascript
Domain
frontend
Issue Type
Fehler
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
2
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
Unter 1 Stunde
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Aktiv
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Klar
Voraussetzungen
JavaScriptGit
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
75