New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Fix: data fetching order #20

Merged

soulchicken merged 4 commits into develop from fix/data-fetching-order

Aug 4, 2024

Member

soulchicken commented Aug 4, 2024

Fix: data fetching order

PR을 한 이유 🎯

데이터를 받아오는 순서랑 랜더링이 꼬여서 화면이 안나올 때가 있었용.

이슈 번호 📎

🐝 Fix: 데이터 패칭 과정이 매끄럽지 않아서 새로고침을 해야 화면이 보여요 #18

변경사항 🛠

데이터를 받아오는 동안 임시로 사용할 loading spinner 컴포넌트를 만들었어요.
vercel 배포에서 우리가 만든 404 페이지가 아니라 vercel 자체 404 페이지가 나오는 상황을 보고 변경했어요.
errorBoundary를 만들어서 에러상황이 나오면 넘어가도록 했어요.
(API로 보내주는 데이터 양식이 바뀌어서 수정했어요.)

soulchicken added 4 commits

August 4, 2024 14:15


          feat: 배포환경에서 404을 띄우기 위해 설정

0b9e939


          feat: loading spinner 컴포넌트

6a8db77


          feat: errorBoundary 컴포넌트

ed37d4c


          feat: 레스토랑 데이터 API 폼 변경 반영

b61d95b

soulchicken merged commit 12dfb67 into develop

1 check passed

Member Author

soulchicken commented Aug 4, 2024

해결 여부에 따라서 이슈는 닫힐수도,,,

soulchicken deleted the fix/data-fetching-order branch

August 10, 2024 03:57

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet