Skip to content

Kimminseo6 week1#7

Merged
bk-git-hub merged 2 commits intokimminseo6from
kimminseo6-week1
Mar 18, 2026
Merged

Kimminseo6 week1#7
bk-git-hub merged 2 commits intokimminseo6from
kimminseo6-week1

Conversation

@kimminseo6
Copy link
Copy Markdown

@kimminseo6 kimminseo6 commented Mar 17, 2026

✅ 제출 정보

  • Week: Week <01>

✅ 체크리스트 (필수)

  • PR의 base branchmain이 아니라 내 GitHub 핸들 브랜치(<handle>)다
  • compare branch가 <handle>-week-xx 형식이다
  • 변경 사항이 이번 주 과제 범위에 해당한다
  • (필요 시) 실행 방법을 적었다
  • (가능하면) 결과 스크린샷/데모 링크를 첨부했다

🧩 구현 내용 요약

React와 JSX를 활용해 Todo 앱의 기본 화면을 구현했습니다.
컴포넌트를 Header, List, Card로 분리해 리스트 형태의 정적 UI를 완성했습니다.

❓ 궁금한 점

없습니다.

결과 스크린샷

image

Copy link
Copy Markdown
Collaborator

@bk-git-hub bk-git-hub left a comment

Choose a reason for hiding this comment

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

민서님 수고하셨습니다~!

전반적으로 깔끔하게 잘 작성해 주셨습니다 그래서 리뷰를 할 부분이 많지 않았어요

다만 제가 커리큘럼 소개할때 말씀드렸던 것처럼 저희 과제는 typescript로 진행되기를 원하고 있습니다. 그래서 다음 주차부터는 jsx 가 아닌 tsx 형식으로 바꿔주시면 좋을 것 같아요.

현재 레포지토리에서 ts를 직접 세팅하시는 방법도 있고, 아니면 vite에서 ts 템플릿으로 다시 시작하는 방법이 있습니다. ts 마이그레이션이 생각보다 복잡해서 새 템플릿에 다음주차과제를 이어하는게 더 쉬울 수도 있습니다.

추가적으로 지금 커밋도 한 커밋에 모든걸 올려주셨는데 작업단위를 쪼개서 커밋하는 연습을 해보면 좋을 것 같아요. 간단하게 가이드를 작성해놨으니까 참고해보시면 좋을 것 같습니다.
https://www.notion.so/Git-Commit-Guide-327793efb80180bcb8f9c357c580a48d

import React from "react";
import ReactDOM from "react-dom/client";
import App from "./App.jsx";
import "../styles.css";
Copy link
Copy Markdown
Collaborator

@bk-git-hub bk-git-hub Mar 18, 2026

Choose a reason for hiding this comment

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

현재 styles.cssindex.html에서도 불러오고 src/main.jsx에서도 import하고 있어서 전역 스타일 진입점은 한 곳으로만 두셔도 될 것 같습니다.`

@bk-git-hub bk-git-hub merged commit 4a4376d into kimminseo6 Mar 18, 2026
3 checks passed
@bk-git-hub bk-git-hub deleted the kimminseo6-week1 branch March 18, 2026 06:47
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