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

[BSVR-225] 리뷰 등록 api 열과 번 모두 nullable 하게 수정 #151

Merged
merged 4 commits into from
Aug 20, 2024

Conversation

pminsung12
Copy link
Collaborator

@pminsung12 pminsung12 commented Aug 20, 2024

📌 개요 (필수)

  • 리뷰 등록 api 열과 번 모두 nullable 하게 수정

🔨 작업 사항 (필수)

  • api endpoint 수정 (seatNumber 삭제)
  • request body에 rowNumber, seatNumber 추가
  • 서비스 로직 수정
  • review entity from 메서드 수정
  • baseReviewResponse 수정
  • create review service .java 리팩토링
    • ReviewDataProcessor 생성
    • MemberLevelProcessor 생성
    • ReviewCreateionProcessor 생성

⚡️ 관심 리뷰 (선택)

  • 특별히 확인 받고 싶은 내용을 적어주세요.

🌱 연관 내용 (선택)

  • 리뷰를 만들 때 여러 엔티티를 조회해 오는데, 지금까지는 요청에서 항상 정확한, 실재하는 id값만이 온다고 가정을 했다. 하지만, 예기치 않은 예외사항에 따른 에러처리가 필요할 것 같은데 여기서 해주는게 맞는지 확신이 서지 않는다..

image


💻 실행 화면 (필수)

image

  • local에서 seatNumber만 null인 경우, seatNumber+rowNumber 둘 다 null 인경우, 둘 다 null이 아닌 경우 다 잘 저장 되는 것을 확인

image

  • 리팩토링 진행한 admin까지 잘 작동하는 것 확인 완료

image

@pminsung12 pminsung12 added ♻️ Refactor 코드 리팩토링 및 구조 개선 ✨ Feature 기능 개발 labels Aug 20, 2024
@pminsung12 pminsung12 self-assigned this Aug 20, 2024
Copy link

github-actions bot commented Aug 20, 2024

Test Results

45 tests  ±0   45 ✅ ±0   0s ⏱️ ±0s
16 suites ±0    0 💤 ±0 
16 files   ±0    0 ❌ ±0 

Results for commit e85a3e1. ± Comparison against base commit 1b3b4e1.

♻️ This comment has been updated with latest results.

@pminsung12 pminsung12 merged commit a911ef2 into main Aug 20, 2024
5 checks passed
@pminsung12 pminsung12 deleted the feat/BSVR-225 branch August 20, 2024 15:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
✨ Feature 기능 개발 ♻️ Refactor 코드 리팩토링 및 구조 개선 size/M
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant