Skip to content

Refactor code review feedback#3

Open
SongHanSol1140 wants to merge 22 commits intoca1af:mainfrom
SongHanSol1140:refactor_code_review_feedback
Open

Refactor code review feedback#3
SongHanSol1140 wants to merge 22 commits intoca1af:mainfrom
SongHanSol1140:refactor_code_review_feedback

Conversation

@SongHanSol1140
Copy link
Copy Markdown

No description provided.

2. 간단한 유저 도메인 추가
3. 티켓생성 TicketCreationDto에 판매자명 추가
4. 가격수정 도메인 비즈니스 규칙 추가(유저확인)
티켓 상태 변경 메서드 추가
딜에 바코드 추가(무슨 티켓을 거래하는지 딜이 알 수가 없음)
거래완료(dealComplete) 추가
딜이 완료되어 판매종료(SOLD)된 티켓 재판매 할수잇도록 수정
각 도메인 서비스 DTO 추가
getter 대신 private set 사용
티켓타입체크 model에서 ApiClientResolver로 이동
패키지 경로 수정
getter 대신 private set 사용
티켓타입체크 model에서 ApiClientResolver로 이동
패키지 경로 수정
refactor(ticket): HALF 상수를 PERFORMANCE_DAY_SALE_POLICY 변경
refactor(ticket): 바코드 길이 매직넘버를 BARCODE_LENGTH 상수로 분리
refactor(ticket): 불필요한 reSale 함수 삭제
feat(ticket): applySellerOfferPrice에 만료 티켓 자동 만료 처리 추가
refactor(ticket): @transactional 변경 감지를 활용하여 불필요한 save() 제거
refactor(api): API 엔드포인트를 restful 규칙에 맞게 변경
refactor(deal): getDealStatus 함수 삭제 후 private set으로 대체
feat(TicketTest): 테스트케이스 추가 + 수정
feat(User): 테스트 추가
feat(KakaoPaymentGateWayTest): 테스트 추가
refactor(UserRepository): 오브젝트에서 @repository 클래스로 변경(레퍼지토리 jpa스타일 통일, DI적용)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant