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

✨ 알림함 백앤드 구현 #113

Merged
merged 11 commits into from
Dec 18, 2024
Merged

✨ 알림함 백앤드 구현 #113

merged 11 commits into from
Dec 18, 2024

Conversation

yunochi
Copy link
Member

@yunochi yunochi commented Dec 18, 2024

✨ 알림함 백앤드 구현

  • 종합 알림함 만들기 #109
  • 최근 알림 (최대 200개)까지 받아올 수 있는 API 엔드포인트 추가
  • 웹소켓으로 알림 오도록 구현
  • DB 테이블 추가

- CreateQuestionDto 수정: isAnonymous 로 익명/기명질문 선택
- QuestionService 수정
  - questioner 자체는 무조건 저장 (비로그인의 경우는 ipHash)
  - isAnonymous boolean을 같이 저장
  - question Dto에는 q.isAnonymous ? null : q.questioner 로 조건부 노출 처리
  - 기타: 노트/툿 올리는데 실패해도 롤백하지 않도록 동작 변경
- DB마이그레이션
- API endpoint
- websocket notification name 변경
Copy link
Collaborator

@Squarecat-meow Squarecat-meow left a comment

Choose a reason for hiding this comment

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

LGTM!

@Squarecat-meow Squarecat-meow merged commit 3d7297b into develop Dec 18, 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