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

[FE][Fix][Docs] #112 : Dropdown 순환 참조 버그 수정 및 스토리북 추가 #181

Merged
merged 6 commits into from
Nov 18, 2024

Conversation

effozen
Copy link
Collaborator

@effozen effozen commented Nov 17, 2024

📝 PR 개요

Dropdown 순환 참조 문제 수정 및 스토리북 추가

🔍 변경 사항

  • Dropdown useContext로 인한 순환 참조 버그 수정
  • 스토리북 디렉토리 구조 수정
  • Dropdown 스토리북 작성

✅ 체크리스트 (Checklist)

  • 코드가 빌드 오류 없이 잘 작동하는지 확인
  • 테스트가 통과하는지 확인
  • 스타일 가이드와 일관성을 유지했는지 확인
  • 관련 문서가 업데이트되었는지 확인 (선택 사항)
  • 리뷰어가 이해할 수 있도록 주석이나 설명을 추가했는지 확인

🔄 관련 이슈 (Linked Issues)

#112

📷 스크린샷 및 동영상 (선택 사항)

스크린샷 2024-11-18 오전 4 28 44

📚 참고 자료 (선택 사항)

https://fantasmith.com/dev-lab/frontEnd/troubleShooting/circular-dependencies
https://fantasmith.com/dev-lab/frontEnd/tools/storybook/introduction
https://fantasmith.com/dev-lab/frontEnd/tools/storybook/look-at-storybook
https://fantasmith.com/dev-lab/frontEnd/tools/storybook/circular-dependency-error
https://fantasmith.com/dev-lab/frontEnd/tools/storybook/how-to-use-useContext-on-storybook

@github-actions github-actions bot added the 확인 요청 리뷰어에게 리뷰 요청 PR 날린 상태 (PR 머지 전) label Nov 17, 2024
Copy link
Collaborator

@leedongyull leedongyull left a comment

Choose a reason for hiding this comment

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

LGTM

@effozen effozen merged commit 795b38d into frontend Nov 18, 2024
3 of 7 checks passed
@effozen effozen added 작업 완료 PR이 제대로 되어 approve 된 상태 (PR 받은 두명 중 마지막 사람이 상태 변경) and removed 확인 요청 리뷰어에게 리뷰 요청 PR 날린 상태 (PR 머지 전) labels Nov 18, 2024
@github-actions github-actions bot added the 확인 요청 리뷰어에게 리뷰 요청 PR 날린 상태 (PR 머지 전) label Nov 18, 2024
@effozen effozen deleted the feature/fe/#112-storybook branch December 5, 2024 07:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
작업 완료 PR이 제대로 되어 approve 된 상태 (PR 받은 두명 중 마지막 사람이 상태 변경) 확인 요청 리뷰어에게 리뷰 요청 PR 날린 상태 (PR 머지 전)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants