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

feat: 개발 서버에서 로그인이 가능하게 기능 추가 #193

Merged
merged 5 commits into from
Sep 9, 2024

Conversation

smb0123
Copy link
Collaborator

@smb0123 smb0123 commented Sep 9, 2024

주요 변경사항

리뷰어에게...

  • 기존의 interceptors가 제대로 작동하지 않게 작성되어 있어서 작동이 잘 되게 변경하였습니다. (로컬 스토리지에 accessToken가 있으면 api request에 Authorization에 넣어주었습니다.)
  • 로그인 성공 시 로컬 스토리지에 accessToken 저장하였습니다.
  • 로그아웃 시 로컬 스토리지의 accessToken을 삭제하였습니다.
  • 백엔드 요구사항에 맞게 api 요청의 쿼리 파라미터에 year를 추가하였습니다.
  • 지원서 작성하고 나오는 페이지에서 여기 버튼을 클릭 시 기존에는 고정된 url로 이동하였는데 접속한 url에 따라 유동적으로 변경하였습니다.

관련 이슈

closes #190

체크리스트

  • reviewers 설정
  • label 설정

@smb0123 smb0123 added the feature 기능개발 label Sep 9, 2024
@smb0123 smb0123 self-assigned this Sep 9, 2024
Copy link
Collaborator

@2yunseong 2yunseong left a comment

Choose a reason for hiding this comment

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

고생하셨습니다~
중복된 이슈가 있는 것 같아요
다만 머지해도 상관없을 듯 하니 진행하셔도 될거 같습니다~

#192

Copy link
Collaborator

@geongyu09 geongyu09 left a comment

Choose a reason for hiding this comment

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

좋네요! 이 이슈로 꽤나 많이 고생하신 듯 한데 정말 수고 많으셨습니다..!

@smb0123 smb0123 merged commit d32ae1b into main Sep 9, 2024
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature 기능개발
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[FE] feat: 개발 서버에서 로그인이 가능하게 구현
3 participants