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.

Sign up for GitHub

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

Jump to bottom

[REFACTOR] 코드리뷰 #113

Open
wants to merge 420 commits into
base: main
Choose a base branch
from
Open

[REFACTOR] 코드리뷰 #113

wants to merge 420 commits into from

Conversation

ujiiin
Copy link
Collaborator

@ujiiin ujiiin commented Aug 13, 2024

✒️ 관련 이슈번호

🔑 Key Changes

  1. 내용
    • API 명세서에 있는 담당에 맞춰 코드 리뷰 진행하고, 코드 리뷰 후 명세서에 진행 상황 업데이트 해주세요
    • 검수자는 dev에서 pull받아 검수 후 PR에 comment 달아주세요
    • comment에 모두 답변을 달아야 머지가 가능합니다. 담당자는 코드 리뷰를 반영한 후 comment에 적절한 답변을 달아주세요
    • 코드 수정 내용이 많을 경우 새 이슈 및 브랜치를 파서 작업해주세요
    • dev에 push하거나, 브랜치를 merge하기 전 로컬에서 오류 없이 잘 돌아가는지 꼭 확인해주세요
    • 막아져있긴 하겠지만 main에서 push하거나 PR을 merge하지 않도록 조심해주세요.
    • 항상 현재 내가 어느 브랜치에 있는지 꼭 확인해주세요

📸 Screenshot

hyoinkang and others added 30 commits August 2, 2024 18:47
@hyoinkang
Merge pull request #45 from UMC-ShowHoo/feature/#39
489b514 
[FEAT] 예매 내역 상태 변경 API 구현
@hyoinkang
Merge pull request #50 from UMC-ShowHoo/feature/#46
dbd4cac 
[FEAT] 관람자/공연 찜 기능 구현
@qormoon
feat: 공연장 사진 등록 api 구현
34b55f2
@qormoon
refactor: 공연장 등록 시 공연장 사진 String으로 받아 저장 #53
0c9c0ab
@qormoon
refactor: 공연장 등록 시 계좌번호 추가 #53
f659966
@qormoon
feat: 성수기 대관료 엔티티 생성 #53
dbb9da1
@dldusgh318
[FIX] dev pull (#34)
abfbfe5
@dldusgh318
[Feat] 공연자/공연장 - 대관 양식, 대관 서류 업로드/다운로드 API 구현 (#34)
90ced02
@dldusgh318
[FIX] dev pull (#34)
9b1b004
@dldusgh318
Merge pull request #54 from UMC-ShowHoo/feature/#34
8e6deae 
[FEAT] 공연자/공연장 - 대관 양식, 대관 서류 업로드/다운로드 API 구현 (#34)
@qormoon
feat: 공연장 등록 시 성수기 대관료 저장할 수 있게 추가 #53
a7a1430
@qormoon
feat: 휴무일 엔티티 생성 #53
09dd95c
@qormoon
feat: 공연장 등록에 휴무일 추가 #53
7525742
@qormoon
fix: db 중복 저장 오류 해결 #53
71fd8bd
@hyesuhan
feat: 대관 수락 및 거절
aa38b66
@hyesuhan
Revert "feat 대관 api"
bd4e4f5 
This reverts commit 443dc08.

# Conflicts:
#	src/main/java/umc/ShowHoo/web/spaceApply/controller/SpaceApplyController.java
#	src/main/java/umc/ShowHoo/web/spaceApply/repository/SpaceApplyRepository.java
#	src/main/java/umc/ShowHoo/web/spaceApply/service/SpaceApplyService.java
@hyesuhan
Merge remote-tracking branch 'origin/dev' into dev
5a9b634 
# Conflicts:
#	.github/workflows/gradle.yml
#	src/main/java/umc/ShowHoo/web/audience/entity/Audience.java
#	src/main/java/umc/ShowHoo/web/book/controller/BookController.java
#	src/main/java/umc/ShowHoo/web/book/dto/BookRequestDTO.java
#	src/main/java/umc/ShowHoo/web/book/dto/BookResponseDTO.java
#	src/main/java/umc/ShowHoo/web/book/service/BookCommandService.java
#	src/main/java/umc/ShowHoo/web/space/entity/Space.java
#	src/main/resources/application-prod.yml
@qormoon
refeactor: 가격 조회 api 수정 #53
3951688
@qormoon
feat: 성수기일 때 가격 변화 #53
e546fdd
@dldusgh318
[MODIFY] Book 테이블 결제금액 추가 (#34)
d2b45ab
@dldusgh318
Merge pull request #55 from UMC-ShowHoo/feature/#34
1b9af41 
[MODIFY] Book 테이블 결제금액 추가 (#34)
@hyoinkang
feature : 공연 게시글 조회 API 시그니처 작성
83b313e
@hyoinkang
Merge branch 'dev' of https://github.com/UMC-ShowHoo/ShowHoo-Server i…
e917cb5 
…nto feature/#51
@qormoon
feat: 대관신청하기 결제하기 api 구현 #53
ceb529f
@qormoon
chore: 대관신청하기 api 설명 추가 #53
2998952
@hyoinkang
feature : 공연 게시글 조회 API 서비스 로직 작성
c768134
@qormoon
Merge pull request #57 from UMC-ShowHoo/feature/#53
c7e911e 
[FEAT] 공연장 등록, 공연장 세부정보 조회, 대관신청 추가 구현
@hyoinkang
feature : 공연 게시글 조회 API + 찜 여부 조회 로직 작성
0047551
@hyoinkang
Merge branch 'dev' of https://github.com/UMC-ShowHoo/ShowHoo-Server i…
5991723 
…nto feature/#51
@dldusgh318
[FEAT] 공연자 - 공연 준비: 예매자 관리 API 구현 (#56)
a40e2f7
dldusgh318 and others added 30 commits August 23, 2024 10:58
@dldusgh318
Feature/#124 (#156)
186c890 
* [REFECTOR] 스웨거 설명 추가

* [FEAT] spaceApply, shows 일대일 연관관계 추가

* [FEAT] shows에 공연날짜, 디데이 관련 dto,converter 구현 (#124)

* [FEAT]공연 준비 시, 공연날짜 공연디데이 관련 로직 (#124)

* [FEAT]공연장 대여가 승인되지않은 경우 에러핸들링(#124)

* [FIX] 코드 정리 (#124)

* [FIX] 공연자 공연 준비 -공연 등록: rentalFile등록 시에 공연 id가 생성되므로, 공연 정보 set하는 방식으로 구현 (#124)

* [FIX] 공연장 공연 준비 - rentalFile 등록 시, 공연id 생성

* [FIX] rentalFile 연관관계 수정

* [FIX] 공연 포스터 오류 수정 (#124)

* [FIX] rentalFile spaceApply연관관계 삭제(이미 shows-spaceApply연관관계 있음)  (#124)

* [FEAT] 공연자 공연준비 - 예매자 예매 상태 넘기기 (#124)

* [FIX] 공연장 공연준비 - 큐시트 작성API 엔드포인트 수정 (#124)

* REFACTOR] 공연자 공연준비 - 큐시트 작성: spaceApplyId로 shows 큐시트파일 저장 (#124)

* [REFACTOR] performerProfileId 수정(#124)
@ujiiin
[FIX] spaceApply, shows 연관관계 수정
dc9c347
@ujiiin
[FIX] spaceApply service 수정
fc185a8
@dldusgh318
[REFACTOR] 공연자 공연준비 시 큐시트 폼 다운 엔드포인트 변경(#124) (#157)
6f1785b 
* [REFECTOR] 스웨거 설명 추가

* [FEAT] spaceApply, shows 일대일 연관관계 추가

* [FEAT] shows에 공연날짜, 디데이 관련 dto,converter 구현 (#124)

* [FEAT]공연 준비 시, 공연날짜 공연디데이 관련 로직 (#124)

* [FEAT]공연장 대여가 승인되지않은 경우 에러핸들링(#124)

* [FIX] 코드 정리 (#124)

* [FIX] 공연자 공연 준비 -공연 등록: rentalFile등록 시에 공연 id가 생성되므로, 공연 정보 set하는 방식으로 구현 (#124)

* [FIX] 공연장 공연 준비 - rentalFile 등록 시, 공연id 생성

* [FIX] rentalFile 연관관계 수정

* [FIX] 공연 포스터 오류 수정 (#124)

* [FIX] rentalFile spaceApply연관관계 삭제(이미 shows-spaceApply연관관계 있음)  (#124)

* [FEAT] 공연자 공연준비 - 예매자 예매 상태 넘기기 (#124)

* [FIX] 공연장 공연준비 - 큐시트 작성API 엔드포인트 수정 (#124)

* REFACTOR] 공연자 공연준비 - 큐시트 작성: spaceApplyId로 shows 큐시트파일 저장 (#124)

* [REFACTOR] performerProfileId 수정(#124)

* [REFACTOR] 공연자 공연준비 시 큐시트 폼 다운 엔드포인트 변경(#124)
@qormoon
chore: space테이블에 위도 경도 추가
c3c375e
@ujiiin
Revert "[FIX] spaceApply service 수정"
de133c6 
This reverts commit fc185a8.
@ujiiin
Revert "[FIX] spaceApply, shows 연관관계 수정"
f7f4217 
This reverts commit dc9c347.
@ujiiin
Merge branch 'dev' of https://github.com/UMC-ShowHoo/ShowHoo-Server i…
27becea 
…nto dev
@ujiiin
[FIX] spaceApply, shows 연관관계 수정
c980166
@hyoinkang
modify : 입장 관리 API 조회 방식 변경 (#159)
69d5ab9
@ujiiin
[FIX] DB create
6b62c13
@ujiiin
Merge branch 'dev' of https://github.com/UMC-ShowHoo/ShowHoo-Server i…
b58e53e 
…nto dev
@ujiiin
[FIX] DB update
0605df2
@hyoinkang
chore : redirectUri 변경
d2f6bf4
@ujiiin
[FIX] yml 설정
878160e
@ujiiin
Merge branch 'dev' of https://github.com/UMC-ShowHoo/ShowHoo-Server i…
3c06c94 
…nto dev
@ujiiin
[FIX] yml 설정
a3e87e6
@hyoinkang
chore : redirectUri 변경
3cb9470
@hyoinkang
Merge branch 'dev' of https://github.com/UMC-ShowHoo/ShowHoo-Server i…
134df5d 
…nto dev
@ujiiin
[ci] ci/cd오류
a72633e
@ujiiin
Merge branch 'dev' of https://github.com/UMC-ShowHoo/ShowHoo-Server i…
a7c938d 
…nto dev
@ujiiin
redirect-uri 변경
348c467
@hyoinkang
chore : redirectUri 변경
67f84de
@dldusgh318
[FEAT] 전체 예매자 조회 (#160)
e7016a9 
* [REFECTOR] 스웨거 설명 추가

* [FEAT] spaceApply, shows 일대일 연관관계 추가

* [FEAT] shows에 공연날짜, 디데이 관련 dto,converter 구현 (#124)

* [FEAT]공연 준비 시, 공연날짜 공연디데이 관련 로직 (#124)

* [FEAT]공연장 대여가 승인되지않은 경우 에러핸들링(#124)

* [FIX] 코드 정리 (#124)

* [FIX] 공연자 공연 준비 -공연 등록: rentalFile등록 시에 공연 id가 생성되므로, 공연 정보 set하는 방식으로 구현 (#124)

* [FIX] 공연장 공연 준비 - rentalFile 등록 시, 공연id 생성

* [FIX] rentalFile 연관관계 수정

* [FIX] 공연 포스터 오류 수정 (#124)

* [FIX] rentalFile spaceApply연관관계 삭제(이미 shows-spaceApply연관관계 있음)  (#124)

* [FEAT] 공연자 공연준비 - 예매자 예매 상태 넘기기 (#124)

* [FIX] 공연장 공연준비 - 큐시트 작성API 엔드포인트 수정 (#124)

* REFACTOR] 공연자 공연준비 - 큐시트 작성: spaceApplyId로 shows 큐시트파일 저장 (#124)

* [REFACTOR] performerProfileId 수정(#124)

* [REFACTOR] 공연자 공연준비 시 큐시트 폼 다운 엔드포인트 변경(#124)

* 전체 예매자 조회 (#124)
@hyoinkang
chore : 위도 경도 추가
9d350e2
@hyoinkang
chore : redirectUri 변경
68d2478
@ujiiin
[FIX] shows, spaceApply 연관관계 수정
0b9f712
@ujiiin
Merge branch 'dev' of https://github.com/UMC-ShowHoo/ShowHoo-Server i…
c5f469a 
…nto dev
@ujiiin
[FIX] shows, spaceApply 연관관계 수정
f4a99b7
@hyesuhan
[ifx] (#161)
380835c 
* [FIX] 대관 날짜로 공연장 확인 및 shows 확인

* [FIX] 대관 날짜로 공연장 확인 및 shows 확인
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@hyoinkang hyoinkang Awaiting requested review from hyoinkang

@hyesuhan hyesuhan Awaiting requested review from hyesuhan

@dldusgh318 dldusgh318 Awaiting requested review from dldusgh318

@qormoon qormoon Awaiting requested review from qormoon

At least 1 approving review is required to merge this pull request.

Assignees

@ujiiin ujiiin

@hyoinkang hyoinkang

@hyesuhan hyesuhan

@dldusgh318 dldusgh318

@qormoon qormoon

Labels
♻️ refactor Good for newcomers
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
5 participants
@ujiiin @hyoinkang @qormoon @hyesuhan @dldusgh318