Skip to content

fix: alarm 문제 수정 #364

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
Merged
merged 2 commits into from
Feb 19, 2025
Merged

fix: alarm 문제 수정 #364

merged 2 commits into from
Feb 19, 2025

Conversation

@kbyunghoon
Copy link
Collaborator

@kbyunghoon kbyunghoon commented Feb 19, 2025 •
edited
Loading

🔧 어떤 작업인가요?

  • 개수 조회 시 유저가 확인하지 않은 데이터만 반환하도록 수정
  • SSE 장시간 알람이 없을 경우에 대한 처리

#️⃣ 연관된 이슈

#363

💡 리뷰어에게 하고 싶은 말

🙏 아래 내용이 모두 충족 되었는지 확인해주세요 🙏

  • PR 이전 dev 브랜치 병합 하셨나요?
  • PR 이전 빌드 테스트 정상적으로 성공했나요?
  • PR 상세내용이 충분히 기재 되었나요?
  • PR 리뷰어, 할당자, 라벨, 프로젝트 확인

@kbyunghoon kbyunghoon added the 🐛 Fix 버그 수정 label Feb 19, 2025
@kbyunghoon kbyunghoon requested a review from tomatozil February 19, 2025 11:01
@kbyunghoon kbyunghoon self-assigned this Feb 19, 2025
@kbyunghoon kbyunghoon linked an issue Feb 19, 2025 that may be closed by this pull request
[fix] 알람 문제 수정 #363
Closed
2 tasks
tomatozil
tomatozil approved these changes Feb 19, 2025
View reviewed changes
Copy link
Collaborator

@tomatozil tomatozil left a comment

Choose a reason for hiding this comment

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

리슨투마 핱빝

@kbyunghoon kbyunghoon merged commit e9f8252 into develop Feb 19, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@tomatozil tomatozil tomatozil approved these changes

Assignees

@kbyunghoon kbyunghoon

Labels

🐛 Fix 버그 수정

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

[fix] 알람 문제 수정

3 participants

@kbyunghoon @tomatozil