Closed p-acid closed 11 months ago
작업 범위에 대해 간략하게 작성합니다.
MVP 배포 이후 자체 QA를 진행하여 개선 작업을 진행합니다.
리뷰어가 중점적으로 봐야 하는 부분을 바로 알 수 있도록 변경된 내용을 나열합니다.
로그인 로딩 상태(isLoading) 변경 분기 수정
isLoading
finally
변경 전
https://github.com/cyclops-operation/switter/assets/87939521/80ba2ba4-9885-488b-884d-e358dd392b30
https://github.com/cyclops-operation/switter/assets/87939521/11fe376d-d815-4bc8-8afc-3c0ba9469678
getDynamicRoute
queryClient.invalidateQueries
then
async/await
invalidateQueries
Promise.all()
env
PR에 대한 추가 설명이나 작업하면서 고민이 되었던 부분 등
전반적으로 논의가 필요한 부분들을 작성했습니다. "이게 맞다" 라기 보다는 논의가 필요한 부분을 수정해봤다고 생각해주시면 감사하겠습니다.
@p-acid 메인 로딩화면에서 로딩 이미지 z-index도 확인 부탁드릴게요!
🛠️ 작업 내용 (Content)
MVP 배포 이후 자체 QA를 진행하여 개선 작업을 진행합니다.
📝 상세 설명
UI/UX 개편
로그인 로딩 상태(
isLoading
) 변경 분기 수정finally
분기에 존재하여 요청이 마무리 된 이후 리다이렉트 되기 전 화면이 로딩 상태가 아닌 것을 실패한 경우에만 로딩 상태를 변경하는 것으로 변경.isLoading
상태를 변경할 필요가 없음변경 전
https://github.com/cyclops-operation/switter/assets/87939521/80ba2ba4-9885-488b-884d-e358dd392b30
https://github.com/cyclops-operation/switter/assets/87939521/11fe376d-d815-4bc8-8afc-3c0ba9469678
getDynamicRoute
) 생성queryClient.invalidateQueries
의then
처리 일괄 변경async/await
형태로 변경then
콜백 반복으로 가독성을 낮춘다고 생각합니다invalidateQueries
처리 방식 변경async/await
을 활용하는 것보다 더 좋다고 생각하여Promise.all()
형태로 수정하였습니다.env
문제를 해결하고자 다른 플랫폼을 활용하고자 합니다.⚙️ 기타 사항
전반적으로 논의가 필요한 부분들을 작성했습니다. "이게 맞다" 라기 보다는 논의가 필요한 부분을 수정해봤다고 생각해주시면 감사하겠습니다.
🚨 Merge 전 필요 작업 (Checklist before merge)