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.

Sign up for GitHub

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

Jump to bottom

Review #1

Open

4 tasks done

ddongule opened this issue Apr 20, 2020 · 1 comment

Member

ddongule commented Apr 20, 2020 •

edited by nyeok98

Loading

😉잘한 점

프로젝트와 그 안의 앱들을 잘 만들어 냈습니다.
프로젝트와 앱들을 url을 통해 잘 연결했습니다.
프로젝트 안의 base.html을 사용해 템플릿을 잘 확장시켜 반복되는 코드를 줄였습니다.
다양한 html 태그들과 알맞은 css를 사용해 유저가 보기 쉽게 프로필 페이지를 만들었습니다.
이미지와 글을 적절히 활용해 알찬 내용의 자기소개를 써주셨습니다.

😎개선 사항(같이 배울 내용이 포함되어 있어요!)

commit message를 조금 더 내용 위주로 써 보는 연습을 하셔도 좋을 것 같습니다.
html 태그들과 div 태그를 적절히 혼합해 사용해 보는 것도 좋을 것 같습니다. 혼자 검색도 해보시고 또 같이 배워보아요!
이미지를 src="" 처럼 url로 가져오는 것이 아니라 프로젝트 내에 넣은 후, 그것을 불러오는 코드를 같이 공부해봅시다!
css 코드를 html 문서가 아닌 새로운 파일로 빼내 link 시키는 법을 배워봅시다.

🥳만드느라 수고하셨어요!

Collaborator

nyeok98 commented Apr 25, 2020

@ddongule
모든 프로젝트에 꼼꼼히 달아주셔서 정말로 감사합니다! 말씀해주신 점 위주로 앞으로 개선해 나가 보겠습니다!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment