perf: [TA-179] 병렬 API 호출로 MyPage 로딩 성능 개선 (Promise.all)#91
Merged
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
src/pages/MyPage/index.tsx에서 순차 실행되던 독립적인 API 호출을Promise.allSettled로 병렬화getUserInfo/getPhotographerInfo,getSavedPromotionList,getSavedPhotographerList3개 API를 동시 요청으로 변경Promise.allSettled사용으로 일부 API 실패 시에도 나머지 결과 정상 처리ignore플래그 패턴)Changes
Promise.all→Promise.allSettled로 변경: 저장 목록 API 실패가 프로필 렌더링을 막지 않음ignore플래그로 unmount 또는 deps 변경 시 stale state 업데이트 방지console.error로 명시적 에러 로깅Test plan
Closes #11