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주차]Instagram-Server-Clone(fuirian) #1

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

Conversation

Copy link
Contributor

@KoSeonJe KoSeonJe left a comment

Choose a reason for hiding this comment

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

ERD도 함께 올려주시면 좋을 것 같습니다. 그리고 PR 제목에 본인 이름도 추가해주시면 좋을 것 같아요

먼저 기능 단위로 커밋을 쪼개는 습관을 들이는 것이 필요할 것 같습니다. 커밋을 쪼개는 기준은 개인마다 기준이 다르나 만약 하나의 기능을 커밋을 쪼갠다면 그 기준을 최대한 통일되게 지켜주시는 것이 좋은 것 같습니다.

로그인 구현은 선행 공부가 필요한 부분이 많아서, 먼저 CRUD를 많이 구현해보고, 다른 기능적인 부분이 담긴 API도 구현을 많이 해본 뒤 익숙해지면 그 때 해도 좋을 것 같습니다.

@fuirian fuirian changed the title [1주차] [1주차]Instagram-Server-Clone(fuirian) Jul 10, 2024
@fuirian fuirian requested a review from KoSeonJe July 11, 2024 01:18
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