lyqht/Octokit-lite

Add Error Boundary for handling request errors

Aberta

#10 aberto em 30 de set. de 2022

Ver no GitHubGuia do colaborador
 (13 comentários) (0 reação) (0 responsável)TypeScript (11 forks)auto 404
good first issue

Métricas do repositório

Stars
 (17 estrelas)
Métricas de merge de PR
 (Métricas PR pendentes)

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.

Guia do colaborador

Direção de pesquisa
Identifique o componente raiz onde um error boundary pode ser colocado. Use o padrão ErrorBoundary do React para capturar erros de rede e exibir uma notificação toast no canto superior direito do aplicativo.
Pilha de tecnologia
typescriptreact
Domain
frontend
Tipo Issue
Funcionalidade
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
2
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
1-3 horas
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
Ativo
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Claro
Prerequisites
ReactTypeScript
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
75