search

2022NewKrew / fe-w4-issue-tracker

0 stars 0 forks source link

[brad.ley] 이슈 트래커 미션 PR - 4 #84

Closed 701b closed 2 years ago

701b commented 2 years ago

:heavy_check_mark: 체크 리스트(PR 올리기 전 아래 내용을 확인해 주세요)

:pencil2: PR 내용

eslint, prettier 설정

기존 IDE(WebStorm) 자체 기능을 사용하다가 여러줄 ternary operation이 정렬이 이쁘게 안되어서 설치하였습니다. eslint는 자바스크립트에 대해서는 IDE 자체 기능이 더 강력하다고 생각해서 그 부분은 끄고, 나머지(react, prettier)에 대해서 설정하였습니다.

진행 상황

  1. Atom Component 구현 거의 완료
  2. 기존 코드 리팩토링 (완벽하게 끝내고 지나가고 싶은 마음에 여기서 시간을 엄청 많이 쓰는 것 같습니다.)
  3. 결국 하던 것 멈추고 IssueList 페이지를 우선 구현해보려고 하는데, 여러 시행착오를 겪다가 우선 전체 템플릿을 완성한 후, 동적인 기능을 구현하는 것이 좋다고 생각하여 이런 방향으로 가고 있습니다.

:guitar: 기타 링크 및 참고사항

부스터샷 맞은 이후로 컨디션이 안 좋아서 커밋까지 신경쓰지는 못했습니다...

701b commented 2 years ago

고생하셨습니다 Brad! propTypes 실제 값이랑 안맞아서 경고뜨는게 조금 있는 것 같아요 ㅎ

PropTypes를 수정해서 모든 경고를 없앴습니다!

쭉 훑어보면서 느낀건데, 지금 시간이 얼마 남지도 않았고, 프로젝트 목표가 주어진 기간보다 좀 큰 스케일이여서 구현하는데 힘드실 것 같다는 생각이 드네요!

예전에 리뷰 남긴 내용들이 아직 반영이 안 된 것 같은데, 프로젝트 완성에 목표를 두지 말고, 리뷰 받은 내용들을 적용하거나 좀 더 개선시키는 그런 곳에 포커싱을 맞추는게 어떨까 싶어요 오히려 그게 이번 기간에 더 얻어갈 수 있는게 많을 수 있겠다는 생각이 드네요! (교육, 리뷰 받은 내용들 잘 기억하면, 추후에 개인, 업무 플젝에서 써먹을 수 있으니까요 :) )

넵 IssueList 페이지 구현까지만 하고 이후에는 개선하는 방향으로 해보겠습니다!

리뷰 부분은 alias 말고는 모두 적용한 것 같은데.. 제가 놓친게 있을까요?!

kakao-hugo-0 commented 2 years ago

@701b 아! 요게 제가 리뷰 남겼다고 생각했는데 빠진 부분들이 있네요 ㅎ image 요렇게 props 넘길때 읽기 쉽게 길이가 짧은 -> 긴 순서, 함수는 마지막 등 규칙을 가지고 두시면 더 좋을 거에요!

다른 분기처리된 부분 mapping으로 바꿀 수 있는지 한번 확인 해주시면 좋을 것 같아용

ColorCodeInput부분도 정규식으로 한번 바꿀 수 있는지 체크해보시고 바꿀 수 있으면 좋을 것 같아요~