Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

김산호 과제 제출합니다. #1

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

coral0723
Copy link

Description

  • 어쩌구 저쩌구

Important content

  • 어쩌구 저쩌구

Question

  • 어쩌구 저쩌구

Reference

@leemanjae02
Copy link
Contributor

  • node_modules 폴더가 프로젝트 밖에 위치하고 있네요! 다음에는 확인 후에 pr해주세요!
  1. 컴포넌트를 분리하고 props로 데이터와 함수를 넘겨 동작하게끔 작성하셨네요!
  2. todoList에 사용되는 type도 따로 분리하여 import해서 사용하고 있으신데
//ts.config

{
  "compilerOptions": {
    "types": ["./src/@types"]
  },

위 옵션을 ts.config에 추가하시면 따로 export하지 않아도 해당 프로젝트에서 전역적으로 type을 불러서 쓸 수 있습니다!
3. 타입스크립트에서 type을 명시할 때 type과 interface를 사용할 수 있는데 이 두가지의 차이점을 공부해보시면 좋을 것 같아요!

  • 과제 조건 피드백
  1. 과제 예시 영상에서는 체크가 되거나 삭제되었을 때 오늘 할 일 카운트가 줄어들어야 하는데 그 기능이 빠져있네요..!
  2. 또한 체크박스와 할 일 문구란의 수직 정렬이 안 맞습니다!

Comment on lines 13 to 20
const onClick = () => {
if (todoText.trim() === '') {
alert('등록할 일정을 입력해주세요!');
} else {
addTodo(todoText);
setTodoText('');
}
};
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

이 부분은 addTodo에서 간단히 얼리리턴 방식으로 구현할 수 있을 것 같네요!

if (!todoText.trim()) {
      alert("등록할 일정을 입력해주세요!");
      return;
    }

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants