Skip to content

Conversation

@seyoung8239
Copy link
Collaborator

PR 목적 / 요약

  • 피드백 관련 컴포넌트 이름 및 구조 변경
    • FeedbackArea → FeedbackList
    • FeedbackBox → FeedbackItem
    • FeedbackForm와 FeedbackList를 같은 계층에서 관리
  • feedbackItem의 editable 여부 관리
  • FeedbackItem 컴포넌트 Storybook에 추가

관련 이슈

리뷰받고 싶은 부분 설명

  • feedbackItem, feedbackList 컴포넌트를 storybook으로 문서화 하는게 좀 어려웠습니다.
    • useRef 관련된 부분
    • recoil에서 자료를 가져오는경우
      • recoil에 초기값 세팅
    • recoil 세팅 부분이 까다로울듯하여 일단 feedbackList는 storybook에 포함시키지 않았습니다.

특이사항

  • FeedbackItem의 editable 옵션을 컴포넌트(/components/FeedbackEditBtns)로 추출
  • FeedbackItem에 FeedbackEditBtns를 composition으로 주입
  • editable 여부는 FeedbackList에서 결정

@seyoung8239 seyoung8239 added this to the Week 5 milestone Dec 8, 2022
@seyoung8239 seyoung8239 requested a review from bh2980 December 8, 2022 10:18
@seyoung8239 seyoung8239 self-assigned this Dec 8, 2022
@seyoung8239 seyoung8239 linked an issue Dec 8, 2022 that may be closed by this pull request
5 tasks
@seyoung8239 seyoung8239 merged commit 435b741 into dev Dec 8, 2022
@seyoung8239 seyoung8239 deleted the 138-피드백-관련-컴포넌트-리팩토링-및-문서화 branch December 8, 2022 17:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

피드백 관련 컴포넌트 리팩토링 및 문서화

2 participants