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

수락 카드 공통 컴포넌트 구현 #39

Merged
merged 14 commits into from
Oct 30, 2023
Merged

Conversation

1g2g
Copy link
Member

@1g2g 1g2g commented Oct 29, 2023

⚙️ PR 타입

  • Feature
  • Hotfix

✨ 기능 설명 or 🚨 문제 상황

수락 카드와 수락 카드 그룹을 구현했습니다.

👨‍💻 구현 내용 or 👍 해결 내용

코드

<AllowCard
  player={{ // player 객체
    id: '1',
    nickname: 'JohnDoe',
    profileImageUrl: 'https://example.com/avatar.jpg',
  }}
/>

예시 코드
      <AllowCardGroup>
        <AllowCard
          player={{
            id: 'hi',
            nickname: '자바',
            profileImageUrl:
              'https://i.ytimg.com/vi/scxNCuffq-0/maxresdefault.jpg',
          }}
        />
        <AllowCard
          player={{
            id: 'hi',
            nickname: '자바와 스크립트',
            profileImageUrl:
              'https://i.ytimg.com/vi/scxNCuffq-0/maxresdefault.jpg',
          }}
        />
        <AllowCard
          player={{
            id: 'hi',
            nickname: '자바와 스크립트자바와 스크립트자바와 스크립트',
            profileImageUrl:
              'https://i.ytimg.com/vi/scxNCuffq-0/maxresdefault.jpg',
          }}
        />
      </AllowCardGroup>

🎯 PR 포인트

  • props로 임의로 정한 player를 받고 있습니다. 모델의 type이 정해지는 대로 수정하겠습니다.

📝 참고 사항

  • 다른 브랜치에서 cherrypick 해와서 사용하다가 충돌이 생겼습니다.
  • 때문에 브랜치를 다시 파서 커밋하느라 커밋 순서가 꼬였습니다...

❓ 궁금한 점

  • 다른 브랜치에서 merge 해와서 사용할 때 다른 분들의 file changed 내용이 안 보이게 막을 수 있는 방법이 있나요? file changed를 전에는 안 보이게 했던 거 같은데 방법이 기억이 안 나네요
  • pull 해오면 다음과 같고
image - merge 해오면 아래처럼 되네요 image

@1g2g 1g2g added the 기능 코드의 기능이 추가되거나 바뀌었습니다. label Oct 29, 2023
@1g2g 1g2g added this to the 1차 스프린트 milestone Oct 29, 2023
@1g2g 1g2g self-assigned this Oct 29, 2023
@1g2g 1g2g linked an issue Oct 29, 2023 that may be closed by this pull request
@1eecan
Copy link
Contributor

1eecan commented Oct 29, 2023

🎉 @1g2g 님 랜덤 리뷰어로 당첨되셨습니다! 리뷰를 부탁드립니다. 🙏

@1g2g 1g2g merged commit b84a35a into dev Oct 30, 2023
@1g2g 1g2g deleted the feat/#35-allow-card-component branch October 30, 2023 07:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
기능 코드의 기능이 추가되거나 바뀌었습니다.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

수락카드 공통 컴포넌트 구현
4 participants