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/#361] 공연 수정하기 Step2(스태프&캐스트 리스트) 수정 가능하도록 변경 + 상세 이미지 추가 및 변경 #379

Closed
wants to merge 25 commits into from

Conversation

ocahs9
Copy link
Contributor

@ocahs9 ocahs9 commented Aug 30, 2024

📌 관련 이슈번호

🎟️ PR 유형

어떤 변경 사항이 있나요?

  • 새 기능 추가
  • 버그 수정
  • CSS 등 사용자 UI 디자인 변경
  • 리팩토링

✅ Key Changes

이번 PR에서 작업한 내용을 간략히 설명해주세요

  1. 작업 내용

📢 To Reviewers

  • 기존에 사라졌던 step2 (스태프 리스트, 캐스트리스트)를 수정할 수 있도록 구현했습니다.
  • 상세 이미지를 추가하고, 수정할 수 있도록 수정했습니다.
  • 변경된 api 명세에 맞게 코드를 변경했습니다.
  • 아직 미리보기 뷰에는 상세 이미지 뷰가 없긴 합니다. (추후 작업 예정)

📸 스크린샷

image
image

  • 수정해야할 버그는 다음과 같습니다. 바로 위의 사진처럼, 종종 CORS 에러가 나서 사진 데이터를 못 가져옵니다..
    image

🔗 참고 자료

ocahs9 added 24 commits August 26, 2024 16:33
@github-actions github-actions bot added 🚀 API API ✨ FEAT 기능 구현 labels Aug 30, 2024
Copy link

PR 작성하느라 고생 많았어요!! 라벨 잘 지정되었는지 확인 한 번 해 주기 🫶

@github-actions github-actions bot requested review from imddoy and pepperdad August 30, 2024 12:27
@github-actions github-actions bot requested a review from sinji2102 August 30, 2024 12:27
Copy link

Quality Gate Failed Quality Gate failed

Failed conditions
1 Security Hotspot
35.8% Duplication on New Code (required ≤ 3%)

See analysis details on SonarCloud

Copy link
Contributor

@imddoy imddoy left a comment

Choose a reason for hiding this comment

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

  1. 포스터 수정 가능하도록 해야할 것 같습니다!
    image
  2. 메이커 정보가 미리보기에 보이지 않습니다.
    image
    image
  3. 미리보기에 안내 문구 추가해주세요!
    image
  4. 그리구 가장 중요한건데, 공연 수정이 되는건가요??
image

@@ -135,6 +149,15 @@ export const CustomFileInput = styled.label<{ width?: number; height?: number }>
border-radius: 6px;
Copy link
Contributor

Choose a reason for hiding this comment

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

p2) 여기 CustomFileInput에 flex-shrink: 0추가해주세요!
image
스크롤 넘어가면 깨지고 있어요!

Copy link
Contributor Author

Choose a reason for hiding this comment

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

넵 반영하겠습니다!

Comment on lines 161 to 163
export const PreviewImageWrapper = styled.article<{ width?: number; height?: number }>`
position: relative;
width: ${({ width }) => (width ? width : 10.8)}rem;
Copy link
Contributor

Choose a reason for hiding this comment

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

p2) 여기 PreviewImageWrapper에도 flex-shrink: 0; 추가해주세요!

Copy link
Contributor Author

Choose a reason for hiding this comment

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

넵 반영하겠습니다!

@ocahs9
Copy link
Contributor Author

ocahs9 commented Aug 31, 2024

공연 수정은 되는데, 종종 이미지가 안 보이거나, 가격이 안보이거나, 매니저가 바로 적용이 안되거나.. 하는 문제들이 발생하고 있습니다 ㅜㅜ 이 문제들 발생 원인을 아직 잘 모르겠어서, 더 고민해봐야할 것 같습니다

@ocahs9
Copy link
Contributor Author

ocahs9 commented Aug 31, 2024

우선 #380에 코리 내용 전부 반영해뒀습니다 ~
merge는 #380 에서 하면 되니 해당 PR은 Close 하겠습니당

@ocahs9 ocahs9 closed this Aug 31, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🚀 API API ✨ FEAT 기능 구현
Projects
Status: ✅Todo
Development

Successfully merging this pull request may close these issues.

[ Feat ] 스태프리스트, 캐스트리스트 수정 가능하도록 구현
2 participants