woowacourse-teams / 2024-cruru

복잡했던 리크루팅을 단 하루만에 크루루.
https://www.cruru.kr
18 stars 6 forks source link

fix-fe: 로그아웃 동기화 이슈 및 공고 수정 시 새로고침 안됨 이슈 #927

Closed github-actions[bot] closed 1 month ago

github-actions[bot] commented 1 month ago

목적

api 요청과 관한 여러 이슈를 픽스합니다.

구현 사항

참고사항

로그아웃 동기화 이슈 원인

현재 clubId는 local storage의 값을 useSyncExternalStorage를 사용해 리액트 상태처럼 사용하는 중입니다. useSyncExternalStorage는 탭 간 공유되는 특성이 있어 한 탭에서 local storage를 비워주면 다른 탭에서도 자동으로 변경이 트리거되어 401 페이지로 진입하게 됩니다. 따라서 의도치 않게 다른 탭에서 로그인을 수행해도 계속 local storage가 지워지는 이슈가 있었습니다.

문제 해결

401 페이지는 백엔드에서 사용자의 토큰이 만료되었을 때 401 응답을 던져 이동되어야 하므로 getClubId가 실패했을 때 서버에서 세션이 만료되었을 때와 동일하게 처리하는 것이 부자연스럽다고 느꼈습니다. 따라서 getClubId 안에 있던 ApiError를 던지는 로직을 제거하고, clubId가 사라진 경우엔 home으로 이동하도록 처리했습니다.

아래의 별표줄 밑에 요구사항 ID만 작성해주세요. Prefix 금지!


FIX_DASH_01

closes #926

github-actions[bot] commented 1 month ago

1729745667.028179

github-actions[bot] commented 1 month ago

1729745670.805399

github-actions[bot] commented 1 month ago

1729745685.331579