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

챌린지 참여 시 로그인 필요한 경우 로직 수정 #150 #151

Merged
merged 2 commits into from
Sep 26, 2024

Conversation

joojjang
Copy link
Collaborator

PR

✨ 작업 내용

#150

  • getDynamicPath.login (로그인 페이지로 리다이렉트시키는 함수) 리팩토링도 했습니다.
  • 자세한 내용은 커밋 메시지 참조

✨ 참고 사항


⏰ 현재 버그


✏ Git Close

- getDynamicPath에 export 추가하고, 함수 내부에서 pathname 얻도록 useLocation 대신 window 객체 사용
@joojjang joojjang added 😡 BugFix 버그 수정 ♻️ Refactoring 코드 리팩토링 및 모듈화 진행 labels Sep 26, 2024
@joojjang joojjang self-assigned this Sep 26, 2024
@joojjang joojjang merged commit 1aff109 into develop Sep 26, 2024
@joojjang joojjang deleted the Fix/issue-#150 branch September 26, 2024 14:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
😡 BugFix 버그 수정 ♻️ Refactoring 코드 리팩토링 및 모듈화 진행
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant