search

devhatt / octopost

Octopost is an open source service from Devhat that was created to guide new developers for free
https://devhatt.github.io/octopost/
MIT License
65 stars 45 forks source link

Error Boundary - Issue 383 #449

Closed zoldyzdk closed 3 months ago

zoldyzdk commented 3 months ago

Closes #383

Feature Remoção do componente de ErrorBoundary próprio e adição do componente vindo do Sentry.
Visual evidences :framed_picture:
ErrorBoundary aplicado no Octopost e retornando uma fallback quando encontra erros no app.
![image](https://github.com/devhatt/octopost/assets/90076846/d4466f7e-df63-4ba1-8c96-a448360b30a4)
Checklist - [x] Issue linked - [x] Build working correctly
Additional info Não temos um layout de página de erro genérica pra servir de fallback pro ErrorBoundary.
zoldyzdk commented 3 months ago

Só quero entender uma coisa antes, até onde me consta um error boundary de respeito deveria dar catch() nos erros estourados e não lotar o console de erro. Isso não ta acontecendo pq? @Alecell

Pelo que eu vi e testei isso não é uma característica do próprio ErrorBoundary. Na documentação do React não faz referência a isso e por exemplo o componente já pronto que eles recomendam (react-error-boundary) também mostra os erros no console.

Pesquisando mais a fundo achei essa issue e nela tem essa resposta do próprio Dan Abramov falando que preferiram não inserir essa opção porque pode facilmente "engolir" erros:

image

Também fui testar como o Nextjs fazia esse tipo de error handling mais genérico com o ErrorBoundary, e ele também mostra o erro no console, mesmo implementando o Error.tsx que vai usar do ErrorBoundary pra capturar os erros e mostrar uma fallback ui ao invés de quebrar a aplicação.