lyqht/Octokit-lite

Add Error Boundary for handling request errors

Ouverte

#10 ouverte le 30 sept. 2022

Voir sur GitHubGuide contributeur
 (13 commentaires) (0 réaction) (0 personne assignée)TypeScript (11 forks)auto 404
good first issue

Métriques du dépôt

Stars
 (17 étoiles)
Métriques de merge PR
 (Métriques PR en attente)

Description

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.

Guide contributeur

Direction de recherche
Identifiez le composant racine où placer une boundary d'erreur. Utilisez le modèle ErrorBoundary de React pour intercepter les erreurs réseau et afficher une notification toast en haut à droite de l'application.
Stack technique
typescriptreact
Domaine
frontend
Type d'issue
Fonctionnalité
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
2
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
1-3 heures
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Active
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
ReactTypeScript
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
75