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

[feat] #183 - 공연 등록 시 공연상세이미지 추가 및 상세페이지, 수정페이지에서의 조회 기능 구현과 웹발신 문자내용 수정 #186

Merged
merged 16 commits into from
Aug 23, 2024

Conversation

hyerinhwang-sailin
Copy link
Collaborator

@hyerinhwang-sailin hyerinhwang-sailin commented Aug 22, 2024

Related issue 🛠

Work Description ✏️

  • 공연상세이미지 presignedUrl로 등록
  • 공연 등록 시 공연상세이미지 추가 가능하도록 변경
  • 공연상세페이지의 공연정보 조회에 상세 이미지 포함
  • 수정페이지에서의 공연정보 조회에 상세 이미지 포함
  • 웹발신 문자 내용 수정
  • dev, prod 토큰 관련 환경변수 설정

Trouble Shooting ⚽️

Related ScreenShot 📷

공연상세이미지가 여러장인 공연 생성할 경우
image
해당 공연 상세페이지 조회
image
수정하기에 필요한 공연 상세정보 조회
image

공연상세이미지가 없는 공연 생성할 경우 아래처럼 빈리스트로 요청 보내면 됩니다.
image

Uncompleted Tasks 😅

To Reviewers 📢

Copy link
Member

@hoonyworld hoonyworld left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

고생하셨습니다!

새롭게 공연 등록 시 공연 상세 이미지를 추가하고, 이로 인해 변동되는 사항 확인했습니다.
문제가 되는 코드는 없어보이며 새롭게 생성된 PerformanceImage와 관련해서 코멘트 달아놓았으니 확인 부탁드립니다!

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

이 부분은 dev 서버에서는 ddl-auto: update로 배포가 되어있어 테이블이 새롭게 생기는거에 대한 문제가 없는데, prod 서버에서는 ddl-auto: none으로 배포가 되어있기 때문에, 테이블을 수동으로 추가해야 할 것 같습니다!!

main에 merge전 꼭 체크하시고, 제게 노티 주시면 감사하겠습니다!!

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

확인했습니다!

@hyerinhwang-sailin hyerinhwang-sailin merged commit c2966e2 into develop Aug 23, 2024
1 check passed
@hoonyworld hoonyworld deleted the feat/#183 branch August 24, 2024 09:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
2 participants