react4noobs icon indicating copy to clipboard operation
react4noobs copied to clipboard

feat(docs): add Error Boundaries

Open gabrielduete opened this issue 2 years ago • 2 comments

Closes: https://github.com/he4rt/react4noobs/issues/127

gabrielduete avatar Oct 18 '23 02:10 gabrielduete

Salve, @rosajsx! Vim aqui finalmente resolver essa PR ahuauh!

Valeu demais, mano!! Excelentes pontos! ❤️

  • Adicionei um exemplo de Error Boundaries com componentes de classe e realoquei para a seção de iniciando com React:
    • https://github.com/he4rt/react4noobs/pull/137/commits/4240d1cfd2fe46427343e96327ccde0a5fabd20a
    • https://github.com/he4rt/react4noobs/pull/137/commits/ba336f566157162f30047a88832d175a7b1e35bb
  • No exemplo do método onError deixei uma explicação extra sobre o crash onde o react desmonta a tela:
    • https://github.com/he4rt/react4noobs/pull/137/commits/514199ff84bc8fb8123f561106bf7533b97692ef
  • Adicionei exemplo sobre observabilidade do erro:
    • https://github.com/he4rt/react4noobs/pull/137/commits/2e854a33b99468e97dbdaa86a9cfb431ee7d1295
  • Adicionei um exemplo utilizando o error boundaries para pegar o stack tracer:
    • https://github.com/he4rt/react4noobs/pull/137/commits/2dae98930373207e76ae175d3c3ae08fa7bc1176

gabrielduete avatar Jan 08 '24 07:01 gabrielduete

Fala mano @gabrielduete sempre bom ver teus PR`S, me animam a continuar o repo, hehe!

Mano, novamente muito parabéns! O conteúdo está muito bom, gostei muito da forma como tu foi levando o leitor a chegar a necessidade de usar o ErrorBoundary realmente muito bom.

Fiz só um comentário ali de detalhe mesmo, coisa besta! Ajeitando isso a gente já sobe ele e vamo de dale em todo o resto!

Muito obrigado, hehe!

rosajsx avatar Jan 10 '24 14:01 rosajsx

Salve, grande @rosajsx ! Tudo certo? (Quanto tempo eim, mano, slc! <3)

E... novamente eu deixei cair a PR no esquecimento.. desculpa, mano!

Fiz só um comentário ali de detalhe mesmo, coisa besta! Ajeitando isso a gente já sobe ele e vamo de dale em todo o resto!

Esses ajustes apliquei nesses commits aqui: https://github.com/he4rt/react4noobs/pull/137#issuecomment-1880495516

Acha que falta mais alguma coisa?

gabrielduete avatar Jun 18 '24 21:06 gabrielduete

Fala meu mano @gabrielduete, quanto tempo ein? Hihiihhi

Eai, vamos voltar a finalizar o repo? Achei perfeito todas as alterações, creio eu que não falta mais nada! Ta ótimo hihihii! Aprovado!

rosajsx avatar Jul 04 '24 10:07 rosajsx

Eai, vamos voltar a finalizar o repo?

Lets bora

gabrielduete avatar Jul 04 '24 21:07 gabrielduete