달리 2팀 최종점검

• 마스코트 캐릭터도 ㅋ.ㅋ 서비스에 애정이 느껴져서 좋습니다.

• 최근도 계속 발전시키고 있는 것 같은데 좋네요 ~~

• git flow, merge 전략 (코드리뷰 언급), rebase 등 선택하신 부분 , 어떻게 공유했는지

• tag 버전관리와 이슈드리븐 개발

• 디자인 적으로는 배경색이랑 폼이 비슷해서 잘 들어나지 않아서 구분을 잘 두는 색을 두셨으면 어떨까 합니다.

• 폼 양옆에 패딩도 조금 좁아 보이는 부분이 있어서 조정하면 더 좋을 것 같아요

• 로그인이나 placeholder 한/영 통일되게 맞추면 더 좋을 것 같아요

• 필드랑 에러메시지 간격을 조금 띄우면 더 가독성이 좋을 것 같아요

• 모달이랑 Toast 컴포넌트도 작업하셨는데 에러 메시지 처리를 컴포넌트를 이용해서 보여주면 좋을 것 같아요

• hover, focus 스타일을 주시면 좋을 것 같아요

• 인증공유 회색버튼의 힌글씨 Contrast 이슈 있을 듯 합니다.

${({ isActive }) =>
      isActive ? colors.default.primary500 : colors.secondary500};
  background-color: ${({ isActive }) =>
    isActive ? colors.primary500 : colors.secondary500};


참고
${({ active }) => active && css`
    color: ${colors.primary500]};
    background-color: ${colors.default.primary500};
`

props에 따라 여러 프로퍼티가 달라지게 되는 경우 css를 활용해서 한 번에 적어주시면 좋습니다.
import { css } from '@emotion/react';

<Timer
        style={{ display: filter === FILTER_TIMER ? 'block' : 'none' }}
        setIsAuth={setIsAuth}
/>

스타일 프로퍼티에 없이 이렇게 적어주셔도 될 것 같아요 
{filter === FILTER_TIMER && <Timer setIsAuth={setIsAuth}/>}

   <Container>
      <Sidebar />
      <UserDetailContainer />
    </Container>

<Container style={{ height: '100vh' }}>
      <Sidebar />
      <Contents />
    </Container>

// component Layout  
 <Container style={style}>
      <Sidebar />
      {children}
 </Container>

버전의 형식은 [Major].[Minor].[Patch] 형식으로 한다.

이전 버전과 호환되지 않는 API 변경은 MAJOR 버전 증가

이전 버전과 호환되면서 기능의 변경, 추가된 경우는 MINOR 버전 증가

버그 수정은 PATCH 버전 증가

• 이건 단순히 비즈니스적인 부분 모각코 서비스인데, Study 인증과 차별화된 서비스인지? → 왜 모각코인지 ? 개발자들을 위한 or 모여서 코딩할 때의 편의성 같은 부분도 고려해보시면 좋을 것 같습니다.

• 달리 1팀의 간격이랑 색 같은 부분 자문 구해서 조금 조정하면 좋을 것 같습니다 :D

• CI/CD에 e2e나 test code를 적용해보시는 건 어떨까요? :D