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

[#16] [Feature] 게시글 도메인 구현 #21

Closed
wants to merge 1 commit into from
Closed

Conversation

Anyeon00
Copy link
Collaborator

@Anyeon00 Anyeon00 commented Sep 26, 2024

✨ 구현 기능 명세

  • Post 엔터티 (+ Enum 클래스 - City) 구현
  • Post 리포지토리 구현

✅ PR Point

😭 어려웠던 점

- Post 엔터티 (+ Enum 클래스 - City) 구현
- Post 리포지토리 구현
@s0o0bn
Copy link
Collaborator

s0o0bn commented Sep 26, 2024

앗 형석님 너무 좋은데요!

저희 PR 단위를 이슈 단위로 할 때, 도메인별 기능 단위로 하는 게 좋을 것 같습니다
예를 들어 지금 형석님 하고 계신게 게시판 쪽이니까 게시판 CRUD 자체가 하나의 이슈가 되고,
그 이슈의 태스크 목록에 게시글 등록, 수정, 삭제, 조회 각각 있는 걸루요 ㅎㅎ
그렇게 해서 feat 커밋은 태스크 단위로 해주시면 어떨까요?

길현님 이슈와 비슷하게 해주시면 좋을 것 같습니당

@Anyeon00 Anyeon00 closed this Sep 26, 2024
@Anyeon00
Copy link
Collaborator Author

넵 그럼 레이어단위가 아닌 기능단위로 구현하도록 하겠습니다!

@Anyeon00 Anyeon00 requested review from a team and removed request for s0o0bn, LJW22222, JiwonKKang and Ban-gilhyeon September 27, 2024 02:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[FEATURE] 게시글 기능 구현
2 participants