Closed nuagenic closed 1 year ago
yarn build
라우팅을 위한 페이지는 ex) app/dashboard/page.js로 만들고, 그 안에 담을 component들을 따로 components 폴더를 만들어 관리했습니다. 이런 방식이 맞을까요? 개선 필요한 지점 있다면 알려주시면 감사하겠슴다
현재 IDInput과 PWInput의 경우 각각 다른 컴포넌트로 관리하고 있는데, 겹치는 코드가 많아 하나로 합치는 게 좋을지, 아니면 그대로 둘지 의견 구합니다.
현재 id와 pw 저장하는 useState는 각각 IDInput과 PWInput에 선언되어 있는데, 이게 좋을까요?(왠지 아닐듯). 상위에 선언하고 props로 내려주면 좋을 것 같은데, 어디까지 상위로 올라가는 게 좋을지 몰라 의견 구합니다.
라우팅되는 버튼의 경우, next에서 제공하는 Link 태그로 감싸는 형태로 작업했습니다. 이 때문에 Curations.js에서 key 값을 Link 태그에 줘야 되던데, 이게 좀 찜찜해서... 괜찮은가요?
Curations는 carousel과 card components로 나누지 않고 하나로 퉁쳐서 작업했는데, 나누는 게 좋을지 의견 구합니다.
c.f) PR 기본 템플릿에 |-----|가 하나 더 있어서 표 적용이 자동으로 안 되는 것 같습니다!
요약
관련 이슈
13
체크리스트
PR 달 때
yarn build
로 테스트를 했나요?머지하기 전에
작업 내용
논의가 필요한 사항
라우팅을 위한 페이지는 ex) app/dashboard/page.js로 만들고, 그 안에 담을 component들을 따로 components 폴더를 만들어 관리했습니다. 이런 방식이 맞을까요? 개선 필요한 지점 있다면 알려주시면 감사하겠슴다
현재 IDInput과 PWInput의 경우 각각 다른 컴포넌트로 관리하고 있는데, 겹치는 코드가 많아 하나로 합치는 게 좋을지, 아니면 그대로 둘지 의견 구합니다.
현재 id와 pw 저장하는 useState는 각각 IDInput과 PWInput에 선언되어 있는데, 이게 좋을까요?(왠지 아닐듯). 상위에 선언하고 props로 내려주면 좋을 것 같은데, 어디까지 상위로 올라가는 게 좋을지 몰라 의견 구합니다.
라우팅되는 버튼의 경우, next에서 제공하는 Link 태그로 감싸는 형태로 작업했습니다. 이 때문에 Curations.js에서 key 값을 Link 태그에 줘야 되던데, 이게 좀 찜찜해서... 괜찮은가요?
Curations는 carousel과 card components로 나누지 않고 하나로 퉁쳐서 작업했는데, 나누는 게 좋을지 의견 구합니다.
c.f) PR 기본 템플릿에 |-----|가 하나 더 있어서 표 적용이 자동으로 안 되는 것 같습니다!