[코드리뷰] 이진수 튜터

[01-binary]

Overview

• 홈페이지 진입시 로딩이 오래걸리는데 어떤 이유때문일까요 ?
• 반응형이 부분적으로만 적용이 되어 있는거 같습니다.
• 영상 페이지에서 뒤로가기를 할 경우 페이지가 죽네요 ㅠ

[01-binary]

README에 왜 아무것도 적혀있지 않나요?

[01-binary]

yarn, npm 둘 중 하나만 사용해주세요

[01-binary]

YourRankingVideo/src/page/DetailPage.jsx

Lines 126 to 132 in f2d1c0f

	if (isLoading) {
	return <p>Loading...</p>;
	}
	if (isError) {
	console.log("오류내용", error);
	return <p>Error..!</p>;
	}

컴포넌트를 렌더하는 부분은 아래의 DetailPageWrapdiv의 return하는 코드의 바로 위에 있어야 가독성이 좋지않을까요 ?

[01-binary]

YourRankingVideo/src/API/util.js

Line 2 in f2d1c0f

export const emailRegex = /^[\w-.]+@([\w-]+\.)+[\w-]{2,4}$/g;

해당 파일이 API 폴더에 있어야할 이유가 무엇인가요 ?

[01-binary]

정답이라고는 할 수 없지만 개인적으로 layout관련 파일들은 component폴더 안으로 넣었을 것 같습니다.

[01-binary]

YourRankingVideo/src/common/firebase.js

Lines 6 to 13 in f2d1c0f

	const firebaseConfig = {
	apiKey: "AIzaSyA8kWftgYd9CGiC1hsPtrJ7GF9FYRONU0E",
	authDomain: "ts-yourranking.firebaseapp.com",
	projectId: "ts-yourranking",
	storageBucket: "ts-yourranking.appspot.com",
	messagingSenderId: "599315602931",
	appId: "1:599315602931:web:8a531a4a3534e80094c686",
	};

중요 키들은 .env 파일로 관리해야합니다.