lyqht/Octokit-lite

Add Error Boundary for handling request errors

オープン

#10 opened on 2022/09/30

 (13 件のコメント) (0 件のリアクション) (0 人の担当者)TypeScript (11 件のフォーク)auto 404
good first issue

Repository metrics

Stars
 (17 個のスター)
PR merge metrics
 (PR metrics pending)

説明

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.

コントリビューターガイド