Skip to content
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.

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

[FE] feat: 라우팅 관련 처리 #752

Merged
merged 3 commits into from
Oct 13, 2023
Merged

[FE] feat: 라우팅 관련 처리 #752

merged 3 commits into from
Oct 13, 2023

Conversation

Leejin-Yang
Copy link
Collaborator

Issue

✨ 구현한 기능

  • 상품 상세 페이지에서 로그인 시 이전 페이지로 이동
  • 로그아웃 후 마이페이지 방문시 페이지 에러 수정

📢 논의하고 싶은 내용

x

🎸 기타

  • 컨텍스트로 상품 상세 경로를 저장해 ProductDetailPage에서 set한 값을 AuthPage에서 읽으려 했지만, 카카오 로그인 경로(외부 경로)로 이동 후 돌아와 컨텍스트가 초기화 되더라구요. 그래서 로컬스토리지를 활용해 구현했습니다 ㅎㅎㅎ
  • 아까 저녁에 올라온 이슈도 같이 처리했어요!

⏰ 일정

  • 추정 시간 : 2시간
  • 걸린 시간 : 6시간

@github-actions
Copy link

github-actions bot commented Oct 12, 2023

Test Results

2 tests   2 ✔️  0s ⏱️
1 suites  0 💤
1 files    0

Results for commit 30af103.

♻️ This comment has been updated with latest results.

Copy link
Collaborator

@hae-on hae-on left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

오늘 하루종일 왜 안되냐며 분노하던 황펭에게 approve를 드립니다.
카카오 링크 때문에 App을 나가서 컨텍스트가 초기화된다는 교훈을 얻게 됐네요.
축하드립니다.

Copy link
Collaborator

@xodms0309 xodms0309 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

코멘트 확인해주세요~~ 고생하셨슴다

};
};

export default useLocalStorage;
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

React의 hooks를 사용하지 않으니 custom hook 말고 util로 분리해도 되지 않을까요?
특히 setLocalStorage라는 함수가 useLocalStorage에서 사용된다면 setState 함수로 오해할 수도 있을 것 같아요

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

좋습니다!

Comment on lines 8 to 10
} catch (error) {
return item;
}
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

에러 났을 때 왜 item을 return하는건가요?! 궁금..

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

json 메서드 실행 에러를 잡아주는 친구입니다
만약 문자열을 저장했다면 에러 발생하고 catch로 가서 문자열 그대로를 반환하는 느낌??

@Leejin-Yang
Copy link
Collaborator Author

Leejin-Yang commented Oct 13, 2023

@xodms0309

로컬스토리지 로직 유틸함수로 이동했습니다. 확인해주세요!

Copy link
Collaborator

@xodms0309 xodms0309 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

귯~ 수고하셨어여~~~

@Leejin-Yang Leejin-Yang merged commit 679ee07 into develop Oct 13, 2023
3 checks passed
@Leejin-Yang Leejin-Yang deleted the feat/issue-718 branch October 13, 2023 07:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[FE] feat: 상품 상세에서 로그인 시 해당 상품으로 리다이렉트
3 participants