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

✨ PokemonCard 컴포넌트 구현 #47

Merged
merged 13 commits into from
Aug 19, 2023
Merged

✨ PokemonCard 컴포넌트 구현 #47

merged 13 commits into from
Aug 19, 2023

Conversation

nalsae
Copy link
Contributor

@nalsae nalsae commented Aug 19, 2023 •
edited
Loading

PR Type

  • FEAT: 새로운 기능 구현
  • ADD : 에셋 파일 추가
  • FIX: 버그 수정
  • DOCS: 문서 추가 및 수정
  • STYLE: 포맷팅 변경
  • REFACTOR: 코드 리팩토링
  • TEST: 테스트 관련
  • DEPLOY: 배포 관련
  • CONF: 빌드, 환경 설정
  • CHORE: 기타 작업

Abstracts

  • PokemonCard 컴포넌트 구현

Description

  • 작업 내용
  • PokemonCard 컴포넌트 구현
  • 일부 컴포넌트에 언어 관련 prop 추가 및 적용
  • 타입 파일 전역으로 변경
  • raw 데이터의 타입 정의
  • 잘못된 스타일링 수정
  • 구현 결과 이미지
    image

Discussion

  • 상태 관리 라이브러리 도입 여부
    : 언어 관련 prop을 전역으로 관리할 때 Context API를 사용할지, 클라이언트 상태 관리 라이브러리를 사용할지 고민해야 합니다.
    : 색상 모드 관련 상태 등 추후 추가될 상태들을 고려하면 클라이언트 상태 관리 라이브러리를 도입하는 것도 괜찮아 보입니다.

Close #35

nalsae added 13 commits August 19, 2023 22:35
@nalsae
✏️ Chore : 공통 타입 파일을 전역으로 변경 (My-Pokedex#35)
a6e8b81
@nalsae
✨ Feat : PokemonCard 컴포넌트에 사용할 API 데이터 타입 정의 (My-Pokedex#35)
dd92584
@nalsae
✨ Feat : PokemonCard 컴포넌트 관련 mock 데이터 생성 (My-Pokedex#35)
0f79791
@nalsae
✨ Feat : 유틸 함수 convertTagName 구현 (My-Pokedex#35)
b24a96a 
- 인수로 전달하는 언어에 따라 type을 tag 이름으로 반환하는 함수
@nalsae
✨ Feat : 유틸 함수 processCardsInfo 구현 (My-Pokedex#35)
ecdc6af 
- API로부터 응답 받은 PokemonCard 관련 raw 데이터를 가공하는 함수
@nalsae
✨ Feat : TypeTag 컴포넌트에 언어 관련 prop 추가 (My-Pokedex#35)
f10be08 
- 언어에 따라 tag 이름을 다르게 렌더링하도록 변경
@nalsae
✨ Feat : TypeFilters 컴포넌트에 언어 관련 prop 추가 (My-Pokedex#35)
cc47f5a 
- 하위 컴포넌트인 TypeTag에 lang prop을 전달하도록 변경
@nalsae
✨ Feat : PokemonCard 컴포넌트 구현 (My-Pokedex#35)
51d9860
@nalsae
✨ Feat : PokemonImage 컴포넌트에 나머지 props 추가 (My-Pokedex#35)
456b953 
- 상위의 PokemonCard 컴포넌트에서 전달하는 스타일링을 통합 시 필요
@nalsae
✨ Feat : CatchButton 컴포넌트에 twMerge 적용 (My-Pokedex#35)
f6aac3f 
- 상위의 PokemonCard 컴포넌트에서 전달하는 스타일링을 통합 시 필요
@nalsae
🐛 Fix : Dropdown 컴포넌트의 잘못된 스타일링 수정 (My-Pokedex#35)
0adfe10
@nalsae
✨ Feat : 구현한 PokemonCard 컴포넌트 임시 적용 (My-Pokedex#35)
74d6c71
@nalsae
Merge branch 'feature/pokemonCard' into develop
23b7c3d
@nalsae nalsae added Fix 이슈 해결 Feature 기능 구현 labels Aug 19, 2023
@nalsae nalsae added this to the ✨ 1차 기능 개발 milestone Aug 19, 2023
@nalsae nalsae self-assigned this Aug 19, 2023
@nalsae nalsae linked an issue Aug 19, 2023 that may be closed by this pull request
✨ PokemonCard 컴포넌트 구현 #35
Closed
1 task
@nalsae nalsae merged commit ce5cf2b into My-Pokedex:develop Aug 19, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@nalsae nalsae

Labels
Feature 기능 구현 Fix 이슈 해결
Projects
None yet
Milestone
✨ 1차 기능 개발
Development

Successfully merging this pull request may close these issues.

✨ PokemonCard 컴포넌트 구현
1 participant
@nalsae