- F12 눌러서 PropTypes 에러 체크하고 Required 된거는 꼭 의미 확인, 넘기시고 PR 날려주쎄용
- 이거 안지키면 propTypes 쓰는 의미가.. 없다!!!
- 관련 변경 사항
- Input 컴포넌트에 value prop 삭제 (useForm을 이용하면 value가 없어야 양방향 바인딩 구현 가능함)
- TextField 컴포넌트에 name prop 추가 (Input에는 name이 꼭 있어야 함)
- propTypes에서 null 삭제
- 안 쓰는 prop 삭제 하기
- Input 컴포넌트 error 삭제
- 새로고침해서 오류 아예 없어졌는지 확인하기
- 10 분이상 지연되면 저한테 말씀해주시기
- 반응형은 일단 고려하지 않기. 우리 반응형은 어렵지 않을 것으로 예상됨
- 리뷰를 구냥 받아들이지 마시고 의문을 가져주신다면.. 감쟈
- 커밋을 고려해서 리뷰 반영하지 않을 거지만, 가까운 시점에 반영할거라면/ merge하지않고 열어둔 채로 다른 작업 하심이 더 편할 것 같습니다!(다른 작업과 관련 없는 독립적인 작업일 경우, 다른 작업과 겹쳐서 일단 merge해야 되는 경우라면 다름)
- 제안입니다!!!!
- 리뷰에 캡쳐 넣을까요?!
- 좋아용
- 스토리북 코드 참고해서 코딩하면 좀더 편합니다
- Readme 를 작업해 봅시다
- form을작업하고 있는데, 빨리는 안될 것 같고.. 음.. 한 한두시간 내로 최대한 완성해서 보내보도록하겠습니다.
- 우선순위가 상인 기능들은 구현 목록에 써놔야 좀 아프올의 일정을 서로 한눈에 알기 쉬울 것 같다.