Skip to content

feat: 5주차 세번째 미션 - #53

Open
kimdoyeon1234 wants to merge 33 commits into
dorifrom
week5/Dori-m3
Open

feat: 5주차 세번째 미션#53
kimdoyeon1234 wants to merge 33 commits into
dorifrom
week5/Dori-m3

Conversation

@kimdoyeon1234

Copy link
Copy Markdown
Collaborator

📚 주차 / 미션

  • n주차 n번째 미션

📌 작업 내용

  • 어떤 작업을 했는지 한 줄 요약

✨ 상세 작업 내용


📸 스크린샷


❓ 리뷰어가 알아야 할 사항 / 질문


✅ 체크리스트

  • 기능 정상 작동 확인
  • 불필요한 주석 삭제
  • 해당 주차 키워드 내용 이해

nonshaman and others added 30 commits March 26, 2026 15:50
@kimdoyeon1234 kimdoyeon1234 self-assigned this Apr 29, 2026

@yewon20804 yewon20804 left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

5주차 미션3도 수고 많으셨습니다🎉 이번에는 구글 소셜 로그인을 진행해보셨는데요 !
미션 수행 잘 하셨고, 간단하게 몇가지 코멘트 남겨드리니 확인 부탁드립니다☺️


구글 로그인 URL 환경변수로 분리

window.location.href = 'http://localhost:8000/v1/auth/google/login';

현재 다른 요쳥들은 환경변수를 잘 활용하고 계신데, 하드코딩되어 있어서 이 부분도 환경변수를 활용하여 호출해주시면 일관성 측면에서도, 추후 배포에서도 문제가 없을 것 같습니다 !


토큰 자동 재발급 시 alert 제거

alert("토큰이 만료되어 자동으로 재발급되었습니다! ");

현재 토큰을 재발급 받으면 다음과 같은 알림창이 사용자에게 나타납니다! 확인을 위해 넣으셨을 수 도 있지만, 토큰 재발급과 같은 내용은 사용자 모르게 처리하는 것이 자연스러운 UX입니다! 그래서 추후 작업에서는 제거해주시는 것이 좋습니다 ☺️

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants