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

Dev: Notification, Like, Conversation 기능 보완 및 세부 기능 추가 #4

Merged
merged 74 commits into from
Jul 31, 2024

Conversation

becooq81
Copy link
Member

📕 Issue

📙 작업 내역

구현 내용 및 작업 했던 내역

  • [ Post/Comment/Notification/Conversation/Message ] 작업 내역 1: 컨트롤러는 비즈니스 로직을 처리하지 않는다. 모든 비즈니스 로직은 서비스단에서 처리하며, 이는 DTO 변환과정 또한 포함한다
  • [ Notification ] 작업 내역 2: 알림의 종류는 NotificationType enum(POST, COMMENT, MESSAGE, BADGE)으로 정의한다
  • [ Notification ] 작업 내역 3: 알림 원인을 식별하는 ID는 causedById로 통합한다
  • [ Notification ] 작업 내역 4: 전체 알림 조회 시 알림 읽음 상태 변경 버그 해결
  • [ Like ] 작업 내역 5: 버그 수정으로 포스트 좋아요 기능 추가
  • [ Conversation ] 작업 내역 6: 로그인 멤버의 전체 대화 목록 조회 및 대화 응답에 마지막 메시지 포함 기능 추가

📘 작업 유형

  • [ ✅ ] 신규 기능 추가
  • [ ✅ ] 버그 수정
  • [ ✅ ] 리펙토링
  • 문서 업데이트

📝 PR 특이 사항

PR을 볼 때 주의깊게 봐야하거나 말하고 싶은 점

  • 특이 사항 1: 서비스와 컨트롤러 단의 분리에 대해 고민하면서 코드를 수정했는데, DTO와 VO의 구분이 확실하지 않아 VO가 꼭 필요한지 고민했습니다



…te a new convo and return the existing convo's id
… that the business logic is only dealt with in service layer
… layer so that the business logic is only dealt with in service layer
@jimmy0006 jimmy0006 merged commit 6912726 into master Jul 31, 2024
1 check passed
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