iterative / iterative/cml.dev

Update `Video` `play()`/`pause()` code, possibly clearing some Sentry errors

Aperta
#52 0 commenti 0 reazioni 0 assegnatari Vedi su GitHub
p2-nice-to-have
Lingua principale
TypeScript
Stelle
13
Fork
22
Metriche di merge delle PR
Nessuna PR unita negli ultimi 30g

Descrizione

* `video.play()` and `video.pause()` return a promise and we should place our code in something like a `try` and `catch` block. Not doing so may be causing some errors that were caught by Sentry
* We could also use a `catch` block to take care of other Sentry `video` errors that we don't need to worry about

## Links about issue
* [Info about `video.play() `](https://developer.mozilla.org/en-US/docs/Web/API/HTMLMediaElement/play)
* [Google Developers article about errors](https://developers.google.com/web/updates/2017/06/play-request-was-interrupted)

## Sentry errors that could be resolved
* [CML-DEV-5](https://sentry.io/organizations/iterative/issues/2380260528/merged/?project=5747165&query=is%3Aunresolved)
* [CML-DEV-C](https://sentry.io/organizations/iterative/issues/2393142694/?project=5747165&query=is%3Aunresolved)

Guida per i contributori

Apri la guida per i contributori

Direzione di ricerca

Individua le chiamate a `video.play()` e `video.pause()` nel sito web e consulta prima le indicazioni collegate di MDN e Google Developers. Usa gli errori Sentry collegati per capire quali rifiuti delle promise devono essere gestiti; il lavoro è completo quando gli errori rilevanti vengono gestiti senza introdurre regressioni.

Scritto dal modello di indicizzazione a partire dal testo della issue.

Valutazione

Stack tecnologico
react, typescript
Ambito
frontend, web-dev
Tipo di issue
Bug
Difficoltà
3/5
Tempo stimato
1-2 giorni
Stato di attività
Ferma
Chiarezza
Abbastanza chiara
Idoneità per principianti
35/100

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.