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: 어드민 서비스 TanstackQuery 설정 및 입장 확인, 어드민 로그인 API 연결 #63

Merged
merged 30 commits into from
Jul 31, 2024

Conversation

coggiee
Copy link
Contributor

@coggiee coggiee commented Jun 29, 2024

연관된 이슈

ex) #이슈번호

issue #61

✅ 작업 내용

이번 PR에서 작업한 내용을 간략히 설명 해주세요(이미지 첨부 가능)

  • SSR TanstackQuery 설정
  • API 연결
    • 입장 확인 API 연결
    • 로그인 API 연결
  • 입장 확인 토스트 메시지 추가
  • 로그인 시 /qr-scan 라우트로 이동(middleware 설정)
  • 상단 네비게이션 바에 로그아웃 버튼 추가(기능은 없음)

🚦 특이 사항

주의깊게 봐야하는 PR 포인트 & 말하고 싶은 점

  • API 테스트 완료했습니다.
  • 입장 확인 시, 토스트 메시지를 보여주도록 추가했습니다.(디자인이 전달되지 않아 임시로 토스트로 설정)
  • middleware를 설정하여, 로그인 시 /qr-scan 라우트로 이동합니다.
    • 서버 단(middleware)에서 로그인 유무를 판별하기 위해, 기존 accessToken 관리 방식을 localStorage -> cookie로 변경했습니다.

💬 리뷰 요구사항(선택)


🚨 이슈를 닫아주세요!

ex) close #이슈번호

close #61

@coggiee coggiee added the ✨ feat 새로운 기능 추가 label Jun 29, 2024
@coggiee coggiee requested a review from SonSuBin129 June 29, 2024 11:33
@coggiee coggiee self-assigned this Jun 29, 2024
Copy link

vercel bot commented Jun 29, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

1 Skipped Deployment
Name Status Preview Comments Updated (UTC)
uket-fe ⬜️ Ignored (Inspect) Visit Preview Jul 27, 2024 10:56am

Comment on lines +1 to +7
export type AccountInfo = {
email: string;
password: string;
};

export type LoginRequestParams = {} & AccountInfo;

Copy link
Contributor

Choose a reason for hiding this comment

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

아! 공통되는걸 이렇게 하면 엄청 깔끔해지네요! 👍

Copy link
Contributor

@SonSuBin129 SonSuBin129 left a comment

Choose a reason for hiding this comment

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

코드 전부 확인했습니다! 수고하셨습니다 :)

@coggiee
Copy link
Contributor Author

coggiee commented Jul 4, 2024

추가 전달 사항

  • 회원가입은 따로 API 요청이 아닌, 구글 폼과 같은 형식으로 가입 요청을 보내는 것으로 변경됐습니다! -> 회원가입 함수는 사용하지 않을 것 같아요
  • 가입 요청을 보낼 수 있도록 로그인 페이지에 링크를 추가할 예정입니다!

@SonSuBin129
Copy link
Contributor

아직 디자인이 나오지 않았지만, 임시적으로 "아이디 정규식이 틀렸을때 나오는 경고 메세지"와 "존재하지 않는 계정일때 나오는 경고메세지" 폰트 색상이 통일되면 좋을 것 같습니다!
추가로, "존재하지 않는 계정일때 나오는 경고메세지"가 폰트 크기가 더 커야 좋을 것 같습니당
QR 인식은 잘 되는 것 확인했습니다 :)

@coggiee
Copy link
Contributor Author

coggiee commented Jul 25, 2024

아직 디자인이 나오지 않았지만, 임시적으로 "아이디 정규식이 틀렸을때 나오는 경고 메세지"와 "존재하지 않는 계정일때 나오는 경고메세지" 폰트 색상이 통일되면 좋을 것 같습니다! 추가로, "존재하지 않는 계정일때 나오는 경고메세지"가 폰트 크기가 더 커야 좋을 것 같습니당 QR 인식은 잘 되는 것 확인했습니다 :)

이메일 필드 아래에 나타나는 경고성 메시지는 삭제하는 방향으로 변경했습니다 (디자인에는 표시되지 않고, 로그인 버튼 상단에 나타나는 경고성 메시지와 동일해서요!)
기존 폰트 크기는 디자인 시안과 동일한데, 조금 더 크게 변경했습니다.

@coggiee coggiee merged commit 0cb0b2b into dev Jul 31, 2024
3 checks passed
@coggiee coggiee deleted the feat/#61 branch July 31, 2024 10:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
✨ feat 새로운 기능 추가
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[feat] 어드민 서비스 TanstackQuery 설정 및 입장 확인, 어드민 로그인 API 연결
2 participants