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

강원대 FE _이채연 - 1주차 과제 #91

Open
wants to merge 8 commits into
base: codus1718
Choose a base branch
from

Conversation

codus1718
Copy link

아직 commit message를 남기는 습관이 부족하여 중간중간 commit message convention을 놓쳤습니다. 또한, 2단계 과제를 정확히 이해하지 못해서 과제를 미완성한것 같아서 2단계 자세히 피드백 부탁드리겠습니다..ㅠ

@codus1718 codus1718 changed the title Feat chaeyeon 강원대 FE _이채연 - 1주차 과제 Jul 1, 2024
@JunilHwang JunilHwang changed the base branch from main to codus1718 July 2, 2024 01:45
Copy link

@lazy-sky lazy-sky left a comment

Choose a reason for hiding this comment

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

안녕하세요 채연님, 이번 리뷰를 맡게 된 김하늘이라고 합니다. 잘 부탁드립니다.

먼저 CRA를 이용하여 프로젝트를 시작하게 되셨네요. 리액트로의 첫걸음을 축하드립니다.

스토리북을 처음 사용하셔서 어려우셨군요. 어떤 부분이 어려웠고, 무엇을 시도했었는데 잘 안됐었는지 말씀해주시면 해결하는데 도움을 드리도록 하겠습니다!

@@ -0,0 +1,47 @@
# react-gift-react-foundation
Copy link

Choose a reason for hiding this comment

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

readme와 readme.old 파일을 구분하신 이유가 있으신가요?

Copy link
Author

Choose a reason for hiding this comment

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

README.md 가 원래 있었는데, react를 설치하고 README가 새로 생기면서 원래 있던 README.md가 README.old.md로 바뀌었습니다. 이럴땐 어떻게 하는게 좋을까요..?

Copy link

Choose a reason for hiding this comment

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

cra로 생성된 리드미는 불필요하니 교체해주시면 될 거 같습니다.

import './App.css';

function App() {

Copy link

Choose a reason for hiding this comment

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

앱 컴포넌트가 아무것도 리턴하지 않고 있네요

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