2주차 코드리뷰

[jhYun505]

In GitLab by @jhYun505 on Oct 11, 2024, 17:42

• 카테고리 관리 페이지
• 메인 관리 페이지
• 헤더
• 푸터

[jhYun505]

In GitLab by @jhYun505 on Oct 11, 2024, 20:21

added 2 commits

• 123cc154 - feat: 메인 페이지 생성 (#6)
• d0aa962f - Merge branch 'feat/6-main' into 'dev'

Compare with previous version

[jhYun505]

In GitLab by @yeomhyeseon on Oct 12, 2024, 14:09

Commented on src/pages/Main.js line 12

url 경로를 'images/main_image1.jog'로 작성하여도 React는 자동으로 public 폴더를 기준으로 경로를 해석하기 때문에 이미지가 잘 로드됩니다.

[jhYun505]

In GitLab by @yeomhyeseon on Oct 12, 2024, 14:09

Commented on src/pages/admin/CategoryManagementPage.js line 61

이 부분도 async await을 사용하면 되겠죠~

[jhYun505]

In GitLab by @yeomhyeseon on Oct 12, 2024, 14:09

Commented on src/pages/admin/CategoryManagementPage.js line 112

정렬 옵션을 배열로 선언하고 매핑하여 사용하는 것이 가독성과 관리차원에서 보다 좋아요~

[jhYun505]

In GitLab by @yeomhyeseon on Oct 12, 2024, 14:10

Commented on src/pages/admin/CategoryManagementPage.js line 6

jsconfig.json 파일을 설정하면 상대경로가 아닌 절대경로를 사용하도록 설정할 수 있어요~

{
  "compilerOptions": {
    "baseUrl": "src"
  },
  "include": ["src"]
}

[jhYun505]

In GitLab by @yeomhyeseon on Oct 12, 2024, 14:10

Commented on src/components/Category/CategoryForm.js line 25

        if (category) {
            setName(category.name);
            setDescription(category.description);
            setParentCategory(category.parentCategory ? category.parentCategory.categoryId : '');
            return;
        }

초기화를 했기 때문에 category가 있을 때만 원하는 값으로 선언해주면 됩니다~

[netlify[bot]]

✅ Deploy Preview for elice-holo ready!

Name	Link
🔨 Latest commit	fa1fcd2fd0b39ceb935ffa530a77acfeaea5f578
🔍 Latest deploy log	https://app.netlify.com/sites/elice-holo/deploys/671ca36d2a462c0008cc5e7a
😎 Deploy Preview	https://deploy-preview-62--elice-holo.netlify.app
📱 Preview on mobile	Toggle QR Code... Use your smartphone camera to open QR code link.

---

To edit notification comments on pull requests, go to your Netlify site configuration.

[jhYun505]

In GitLab by @yeomhyeseon on Oct 12, 2024, 14:10

Commented on src/components/Category/CategoryForm.js line 77

화면을 새로고침할 필요없이 category list를 새로 불러오는 callback함수로 받아서 실행하면 됩니다~

[jhYun505]

In GitLab by @yeomhyeseon on Oct 12, 2024, 14:10

Commented on src/components/Category/CategoryForm.js line 76

category 여부로 put, post를 구분하고 이후에 에러처리는 catch에서도 조건 처리를 해서 처리할 수 있겠어요~

[jhYun505]

In GitLab by @yeomhyeseon on Oct 12, 2024, 14:10

Commented on package.json line 46

carousel은 개발중에만 사용하는 모듈이 아닌 배포 후에도 사용해야하기 때문에 dependencies에서 포함해주세요~

[jhYun505]

In GitLab by @yeomhyeseon on Oct 12, 2024, 14:12

일주일동안 고생 많으셨습니다\~!
리뷰 작성하면서 공통되는 내용은 페이지별로 작성하지는 않고 한곳에만 작성해두었어요.
그렇기 때문에 각자 작업한 내용에 대한 피드백 외에도 다른 분들 내용도 함께 확인하시고 작업하실 때 적용해서 작업해 주세요\~ 🙌🏻

[jhYun505]

In GitLab by @jhYun505 on Oct 14, 2024, 14:01

mentioned in commit 3628c532d1583a87aed32dfa8bb4927f1a3d8248