depromeet / ahmatda-web

아맞다 :white_check_mark: 이젠 까먹지 말고 챙기자
https://github.com/depromeet/ahmatda-web#download
7 stars 0 forks source link

에러바운더리 개발 및 적용 #237

Closed hyesungoh closed 1 year ago

hyesungoh commented 1 year ago

🤔 해결하려는 문제가 무엇인가요?

🎉 어떻게 해결했나요?

사실 mutation, query 등에서 던져지는 에러를 핸들링하기 위해 시작했으나 거기서 던져지는 것은 잡아내지 못하더라구요 ! -> 이건 onError에서 핸들링해야 할듯 싶어요!

📚 Attachment (Option)

cloudflare-pages[bot] commented 1 year ago

Deploying with  Cloudflare Pages  Cloudflare Pages

Latest commit: 32d0be4
Status: ✅  Deploy successful!
Preview URL: https://51c5e7c2.12-team3-web.pages.dev
Branch Preview URL: https://issue-236.12-team3-web.pages.dev

View logs

github-actions[bot] commented 1 year ago

Bundle Sizes

Compared against 0f7884d0abb5e1e894ddfa1071a4db82422fa85a

Route: No significant changes found

Dynamic import: No significant changes found

hyesungoh commented 1 year ago

지금 생각해보니 리액트 쿼리에서 던져지는 에러는 쿼리 클라이언트에서 핸들링할 수 있겠네요 ! https://tkdodo.eu/blog/react-query-error-handling#the-global-callbacks


근데 토스트 메세지를 띄우는 행동으로 가정했을 때, 모든 에러가 토스트 메세지를 띄울 필요가 있을까 ... 있는 거 같기도하고 ... 근데 중요도에 따라 토스트 메세지 시간이 달라야되나 ...

암튼 그러한 이유로 따로 핸들링 해볼게요 !

kooku0 commented 1 year ago

지금 생각해보니 리액트 쿼리에서 던져지는 에러는 쿼리 클라이언트에서 핸들링할 수 있겠네요 !

https://tkdodo.eu/blog/react-query-error-handling#the-global-callbacks


근데 토스트 메세지를 띄우는 행동으로 가정했을 때, 모든 에러가 토스트 메세지를 띄울 필요가 있을까 ...

있는 거 같기도하고 ... 근데 중요도에 따라 토스트 메세지 시간이 달라야되나 ...

암튼 그러한 이유로 따로 핸들링 해볼게요 !

저도 예전에 비슷한 고민했었는데요 그렇게 되면 만약 에러 처리가 쿼리에 매우 종속적이게 되어 따로 쓰지 못할수도 있긴해요 만약 따로 안쓰일것 같으면 이렇게 해도 될것 같기도?