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

유저 통합 및 Soft Delete 적용 #45

Conversation

1jeongg
Copy link
Member

@1jeongg 1jeongg commented Oct 7, 2023

한 일

  • 유저 통합(@Inheritance(strategy = InheritanceType.SINGLE_TABLE)): 단일 테이블 전략 사용
  • @SQLDelete를 이용한 soft delete 적용
  • 멘토님 리뷰 기반 리팩토링
  • 결제 API 추가(payments/save, payments/confirm, payments/upgrade)

Todo

  • 테스트 코드 추가
  • 포트폴리오 RepositoryTest 고치기 (현재는 주석처리)

close #43
close #44

@1jeongg 1jeongg added Done 진행 완료 user 유저와 관련된 기능 refactor labels Oct 7, 2023
@1jeongg 1jeongg self-assigned this Oct 7, 2023
- 결제 검증
- 결제 정보 저장
- 유저 업그레이드
- user의 필드인 id와 payed_amount를 Long 타입으로 변경
- validation 검증은 GlobalExceptionHandler에서 진행
- 유저 조회 API에서 userId 추가됨
- 이미지 불러오는 경로가 잘못됨
@Rizingblare Rizingblare merged commit 4ff528a into Step3-kakao-tech-campus:weekly-refactor Oct 7, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Done 진행 완료 refactor user 유저와 관련된 기능
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants