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

[Feature] 로그인 구현 #20

Merged
merged 60 commits into from
Aug 14, 2024
Merged

[Feature] 로그인 구현 #20

merged 60 commits into from
Aug 14, 2024

Conversation

ghdtjgus76
Copy link
Collaborator

🎉 변경 사항

로그인 로직 구현 완료했습니다.

🚩 관련 이슈

🙏 여기는 꼭 봐주세요!

아직 와우 디시 버튼 컴포넌트 변경 사항이 반영되지 않아서 추후에 깃허브 로그인 버튼 아이콘 추가해둘게요!

로그인 관련 로직에서 useRouter 훅 사용해서 라우팅하려고 클라이언트 컴포넌트로 구현한 부분이 많은데, Button 컴포넌트 as props 추가해서 next/link의 Link 컴포넌트 수용할 수 있도록 하면 다 서버 컴포넌트로 변경할 수 있을 거 같습니다!
현영님이 추후에 버튼 리팩토링해주신다는 부분 반영해주시면 저도 반영해둘게요~

Copy link

@ghdtjgus76
Copy link
Collaborator Author

@eugene028
일단 임시?로 해결해뒀습니다!
어제 보내주신 링크도 참고해서 다 해봤는데 안 돼가지고, middleware에 Authorization header 적용하는 방법 써서 해결했습니다
다만 대시보드 api는 미들웨어에서 불러와야 해서, 해당 api만 헤더 같이 넣고 요청하도록 했습니다
다른 api는 따로 헤더 안 넣어도 동작할 거예요

@eugene028
Copy link
Collaborator

@eugene028 일단 임시?로 해결해뒀습니다! 어제 보내주신 링크도 참고해서 다 해봤는데 안 돼가지고, middleware에 Authorization header 적용하는 방법 써서 해결했습니다 다만 대시보드 api는 미들웨어에서 불러와야 해서, 해당 api만 헤더 같이 넣고 요청하도록 했습니다 다른 api는 따로 헤더 안 넣어도 동작할 거예요

오 빠르게 해결하셨군요 감사합니당,,
어제 미들웨어에서 컨트롤하고 있는 Next헤더 뜯어와서 이것저것 봤는데 쿠키도 잘 심기고 host도 재현님이 넣어주신대로 잘 들어가고 있어서 딱히 소득은 없었네유 ㅜ
임시방편으로 이렇게 두고 진짜 이유를 찾아봅시다...😂

Copy link
Collaborator

@eugene028 eugene028 left a comment

Choose a reason for hiding this comment

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

요거 일단 풀받아서 어드민에서도 잘 되는지 확인해볼게요 ~!
일단 어프룹 눌러놓늡니당

Copy link

Copy link

Copy link
Collaborator

@SeieunYoo SeieunYoo left a comment

Choose a reason for hiding this comment

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

수고 하셨어요 👍우선 어푸룹 눌러두겠습니덩

Copy link

@ghdtjgus76
Copy link
Collaborator Author

@eugene028 @SeieunYoo
리뷰 반영 완료했습니다!
다른 분들 api 작업하셔야 하니까 일단 얼른 머지해놓겠습니다~

@ghdtjgus76 ghdtjgus76 merged commit 1226697 into dev Aug 14, 2024
2 of 3 checks passed
@ghdtjgus76 ghdtjgus76 deleted the feature/login branch August 14, 2024 10:28
@ghdtjgus76 ghdtjgus76 linked an issue Aug 14, 2024 that may be closed by this pull request
1 task
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.

[Feature] 로그인 구현
4 participants