lyqht/Octokit-lite

Add Error Boundary for handling request errors

Aperta

#10 aperta il 30 set 2022

Vedi su GitHubGuida contributor
 (13 commenti) (0 reazioni) (0 assegnatari)TypeScript (11 fork)auto 404
good first issue

Metriche repository

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

Descrizione

Description

In both Unfork and TopicSpace, there is no error handling of showing error notification/popup in the case when the network request to API routes fails e.g. in the case of bulk repository delete (#3) or other edge cases.

Suggested Solution

Add a React error boundary to catch all network errors and display it as a simple error popup ui at the right top edge of the app.

Guida contributor

Direzione di ricerca
Identifica il componente radice dove posizionare un error boundary. Usa il pattern ErrorBoundary di React per catturare gli errori di rete e mostrare una notifica toast in alto a destra dell'app.
Tech stack
typescriptreact
Dominio
frontend
Tipo issue
Funzionalità
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-3 ore
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
ReactTypeScript
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
75