pre-onboarding-9th-3-10: A TypeScript repository from yoosion030

원티드 프리온보딩 FE 인턴십 3주차 과제

원티드 프리온보딩 FE 인턴십 3주차 과제물 입니다.

플렉시스 브랜드의 기업 과제를 진행하였습니다.

작업 기간 : 3/13 ~ 3/17 (5일간 진행)

🌟Preview

메인 페이지	필터링 후 메인 페이지

📝Contents

배포 링크

https://pre-onboarding-9th-3-10.vercel.app/

실행 방법

$ git clone https://github.com/wanted-onboarding-10team/pre-onboarding-9th-3-10.git
$ npm i
$ npm start

사용 기술 스택

파일 구조

📦
├─ public
│  ├─ data 
│  └─ └─ mock_data_examples-flexsys.json
├─ src
│  ├─ assets 
│  ├─ components 
│  │  ├─ common 
│  │  ├─  └─ MainLayout.ts
│  │  ├─ chart 
│  │  ├─  └─ CustomTooltip.tsx
│  │  ├─  └─ FilterButton.tsx
│  │  ├─  └─ MainChart.tsx
│  │  └─ index.ts
│  ├─ pages
│  │  └─ MainPage.tsx
│  ├─ router
│  │  ├─ loader 
│  │  │   ├─ mainLoader.ts
│  │  │   └─ rootLoader.ts
│  │  └─ index.tsx
│  ├─ router 
│  │  └─ loader
│  ├─ styles 
│  │  └─ GlobalStyles.tsx
│  ├─ types 
│  │  └─ types.ts
│  ├─ utils/api
│  │  └─ chart.ts
│  ├─ index.js
└─ └─ App.js

✅ Team Rules

1. 커밋 컨벤션

feature	새로운 기능 추가
fix	버그 수정
docs	문서 수정
refactor	코드 리팩토링
style	스타일 변경
chore	패키지 설치 및 빌드 부분 수정

자세한 사항은 .gitmessage.txt 에서 확인 가능합니다

2. Git Flow

조직의 레포지토리를 Fork를 사용하여 협업하는 방식을 채택하였습니다.

기존 각자 브랜치를 나눠서 작업하던 방식에서 바꾼 이유는 아래와 같습니다.

더욱 자유로운 환경에서 개발을 시도해 보기 위함
브랜치 관리를 더욱 깔끔하게 하기 위함

// 이전 방식
- main : 배포 브랜치 
	- develop : 개발 브랜치
		- feature/#[이슈번호] : 각 기능을 개발하는 브랜치
			- feature/#[이슈번호]-팀원이름

3. Prettier

// .prettierrc
{
  "printWidth": 100,
  "singleQuote": true,
  "arrowParens": "avoid",
  "semi": true,
  "tabWidth": 2,
  "endOfLine": "auto",
  "trailingComma": "all",
  "useTabs": false,
  "bracketSpacing": true,
  "jsxSingleQuote": true
}

4. EsLint

// .eslintrc
{
  "env": {
    "browser": true,
    "es2021": true
  },
  "extends": [
    "eslint:recommended",
    "plugin:react/recommended",
    "plugin:@typescript-eslint/recommended",
    "plugin:prettier/recommended"
  ],
  "overrides": [],
  "parser": "@typescript-eslint/parser",
  "parserOptions": {
    "ecmaVersion": 2018,
    "sourceType": "module",
    "ecmaFeatures": {
      "jsx": true
    }
  },
  "plugins": ["react", "@typescript-eslint", "import"],
  "rules": {
    "no-var": "error",
    "prettier/prettier": "error",
    "no-multiple-empty-lines": "error",
    "no-console": [
      "warn",
      {
        "allow": ["warn", "error", "info"]
      }
    ],
    "dot-notation": "error",
    "eqeqeq": "error",
    "no-unused-vars": "warn",
    "react/jsx-uses-react": "off",
    "react/react-in-jsx-scope": "off"
  },
  "settings": {
    "react": {
      "version": "detect"
    }
  }
}

👏 협업 방법

주된 커뮤니케이션 툴로 팀 노션 페이지와 Discord, Figma를 사용했습니다.

기능별로 팀원 개개인의 코드 리뷰
토론을 통해 Best Practice 선정

노션 페이지 / Figma
- 회의를 통해 팀원들의 코드를 분석하고, 그 중에서 Best Practice 정하고, commit message convention, git flow 전략 등 Team Rules를 정하기 위해 활용
Discord
- 팀원들의 의사소통 및 화면 공유를 통한 협업을 위해 활용

🌟 프로젝트 진행 과정

Issue와 PR을 통해 코드 리뷰를 진행하며 프로젝트를 진행하였습니다.

Issue를 정의하고 하루에 한가지의 이슈를 다같이 진행한 후 피드백하는 식으로 동료 협업을 진행하였습니다.

Issue

날짜	제목
3/13	#1 개발환경 세팅
3/14	#2 요구사항 1: 시계열 차트 만들기
	#3 요구사항 2: 호버 기능 구현
	#4 요구사항 3 : 필터링 기능 구현
3/16	Code refactoring

주요 논점

그래프 라이브러리 rechart 선정
- 리액트에 친화적인 라이브러리라는 평 즉, 리액트 컴포넌트처럼 사용이 가능한 점이 가장 큰 장점
- 라이브러리 API 기술서가 이번에 주어진 과제 요구사항에 적합
- npm 다운로드 수가 다른 차트 라이브러리들에 비해 압도적으로 높음 (23년 3월 기준 1.1M+)
- 차트 컴포넌트를 여러가지를 조합하여 사용할 수 있다는 이점
[서영]의 코드로 선정된 이유
- 전반적으로 UI가 보기에 깔끔하였다.
- 원래 Bar와 Area의 값 차이가 컸는데 값 차이를 비슷하게 맞춘 점
- Custom tootip 컴포넌트를 한 것과 특히 다중 필터링 기능을 구현 했던 점
Code Refactoring
- 변수 네이밍
  
  어떤 데이터들인지 가리키는 값을 명확하게 드러나도록, 명확히 의미부여를 하기 위해 논의
  - 받아온 데이터
    - ChartDataResponse → 받아온 데이터라는 의미가 명확한 것 같아 채택
  - 가공된 데이터(차트에서 사용되는 데이터)
    - chartData
  - 필터링 id
    - areaCategory
    - selectedCategory ⇒ 카테고리라는 키워드로 변경
  - 타입
    - ChartData
- 데이터 가공 위치
  - Hook vs loader
    
    Hook으로 쓰기에는 데이터 형변환 하는 것 뿐이라 반복되는 로직을 묶어 하나의 컴포넌트를 만드는 hook과는 맞지 않다고 생각하여 loader를 사용해 데이터 변환 로직 적용
- 파일 구조
  
  pages > MainPage.tsx
  
  components > chart > 차트에 관련된 컴포넌트

참여 멤버

박수완	유시온	이새미	한동룡	홍수민	황서영
@skdoqj	@yoosion030	@shinpanda	@Ryong-E	@hongsoom	@Seo0H

yoosion030/pre-onboarding-9th-3-10