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 : PosterWithDetailInfo 컴포넌트 구현 #166

Merged
merged 13 commits into from
Oct 7, 2024

Conversation

jymaeng1234
Copy link
Collaborator

@jymaeng1234 jymaeng1234 commented Oct 6, 2024

📝 Summary

  • 공연 기본 정보를 표기합니다.
  • 찜하기, url 공유(복사), 리뷰 작성 기능이 있습니다.
  • PosterWithButtons, PDetailInfo,PDetailReviewForm 가 그룹화된 컴포넌트입니다.

✨ Changes

  • PosterWithDetail 컴포넌트 생성
  • PosterWithButtons, PDetailInfo 컴포넌트의 css 수정

🖼️ Screenshots

image

✅ PR Checklist

  • 하나의 목적을 가진 PR입니다.
  • 코딩 컨벤션과 스타일 가이드를 준수합니다. 📌Conventions
  • 불필요한 코드 중복이 없습니다.
  • 컴포넌트의 책임이 단일합니다.
  • 리액트 훅을 올바르게 사용했습니다.
  • 민감한 정보를 포함하지 않았습니다.
  • 불필요한 콘솔 로그나 주석을 포함하지 않았습니다.
  • 컴포넌트 key값에 고유한 값을 할당했습니다.

🔗 References

💬 Comments

  • isNameInclude 값을 true로 설정할 경우, 공연 이름/ 장르가 함께 표기됩니다.
  • PDetailReviewForm 컴포넌트와 PDetailInfo컴포넌트의 스타일 통일 작업이 필요합니다.

@jymaeng1234 jymaeng1234 self-assigned this Oct 6, 2024
@jymaeng1234 jymaeng1234 linked an issue Oct 6, 2024 that may be closed by this pull request
src/components/PosterWithDetailInfo/index.tsx Outdated Show resolved Hide resolved
src/components/PosterWithDetailInfo/index.tsx Outdated Show resolved Hide resolved
src/components/PosterWithDetailInfo/index.tsx Outdated Show resolved Hide resolved
@shlee9999
Copy link
Collaborator

나머지는 Slack에 보내드린 링크 참고하셔서 진행해주세요!

@jymaeng1234
Copy link
Collaborator Author

코멘트 주신 부분 모두 수정했습니다.

@jymaeng1234 jymaeng1234 force-pushed the 146-feature/posterwithdetailinfo-component branch from 73b7809 to 187cdd0 Compare October 7, 2024 01:04
@shlee9999 shlee9999 merged commit 66b4f2f into main Oct 7, 2024
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.

🎯[Feature]PosterWithDetailInfo 컴포넌트 구현
3 participants