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

[HEENDY-61-user-QR-API] 유저용 QR코드 생성 및 memberId UUID로 받기 #17

Merged
merged 9 commits into from
Feb 26, 2024

Conversation

sangeun99
Copy link
Member

구현 사항

  • 회원가입 시 유저의 QR코드 생성, S3에 저장 후 DB 업데이트
  • 유저의 QR코드 조회 API (GET /api/v1/members/qr)
  • 테스트용 QR코드 생성 API (POST /api/v1/members/qr)
  • memberId 타입 전부 String으로 수정
  • member 가입 시 id UUID로 생성하여 저장

참고 사항

  • memberId 타입 전부 String으로 수정한 부분 친구, 이벤트, 쿠폰, 멤버 조회 API 테스트 완료
  • 다른 API의 경우 추가 테스트 필요 @sooyoungh
  • AWS S3 환경변수 Jira 참고하여 세팅

코드리뷰 필요한 부분

  • MemberServiceImple의 joinByOauthId 메소드 수정 부분
  • (디테일하게는 MemberQrService, AwsS3Config 부분)

스크린샷

S3 버킷 스크린샷

image

POSTMAN 스크린샷

image

참고자료

@sangeun99 sangeun99 added 💡 feature 기능 개발 🐞 fix 버그 수정 🐻‍❄️ 연유 @sangeun99 🔥 H-5 5시간 안에 리뷰해주세요 labels Feb 26, 2024
@sangeun99 sangeun99 requested a review from sooyoungh February 26, 2024 05:42
@sangeun99 sangeun99 self-assigned this Feb 26, 2024
@sangeun99 sangeun99 marked this pull request as ready for review February 26, 2024 05:43
@sangeun99
Copy link
Member Author

기존 방식에서는 회원가입 시 시퀀스로 memberId가 결정되기 때문에 먼저 insert 후 id를 받아와 QR코드를 생성하고, 그것을 DB에 업데이트 했음.
하지만 memberId를 UUID로 수정함에 따라 QR코드 생성 후 멤버 데이터를 한번에 insert하도록 수정함

@sangeun99 sangeun99 merged commit 4271aba into dev Feb 26, 2024
2 checks passed
@sooyoungh sooyoungh deleted the HEENDY-61-user-QR-API branch March 1, 2024 11:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🐻‍❄️ 연유 @sangeun99 💡 feature 기능 개발 🐞 fix 버그 수정 🔥 H-5 5시간 안에 리뷰해주세요
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant