lyqht/Octokit-lite

Add Error Boundary for handling request errors

Offen

#10 geöffnet am 30.09.2022

Auf GitHub ansehenContributor Guide
 (13 Kommentare) (0 Reaktionen) (0 zugewiesene Personen)TypeScript (11 Forks)auto 404
good first issue

Repository-Metriken

Stars
 (17 Sterne)
PR-Merge-Metriken
 (PR-Metriken ausstehend)

Beschreibung

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.

Contributor Guide

Research-Richtung
Identifiziere die Root Komponente, in die eine Fehlergrenze eingefügt werden kann. Verwende das ErrorBoundary Muster von React, um Netzwerkfehler abzufangen und eine Toast Benachrichtigung oben rechts in der App anzuzeigen.
Tech Stack
typescriptreact
Domain
frontend
Issue Type
Funktion
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.
1-3 Stunden
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
ReactTypeScript
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
75