Skip to content

[refactor] 더티 체킹 제거 및 명시적 저장 방식으로 리팩토링#272

Merged
pshshsh merged 1 commit intodevfrom
feature/waitlist-notification
May 2, 2025
Merged

[refactor] 더티 체킹 제거 및 명시적 저장 방식으로 리팩토링#272
pshshsh merged 1 commit intodevfrom
feature/waitlist-notification

Conversation

@pshshsh
Copy link
Copy Markdown
Contributor

@pshshsh pshshsh commented May 2, 2025

  • 알림 단건 읽음 처리 시 save 명시 호출
  • 전체 알림 읽음 처리 루프 내 save 추가
  • 알림 수신 여부 변경 시 userRepository.save() 명시 저장
  • Vacancy 알림 처리 시 waitlistRepository.save() 저장 처리

🔗 Issue Number

close #271

📝 작업 내역

  • 알림 단건 읽음 처리 시 save 명시 호출
  • 전체 알림 읽음 처리 루프 내 save 추가
  • 알림 수신 여부 변경 시 userRepository.save() 명시 저장
  • Vacancy 알림 처리 시 waitlistRepository.save() 저장 처리

- 알림 단건 읽음 처리 시 save 명시 호출
- 전체 알림 읽음 처리 루프 내 save 추가
- 알림 수신 여부 변경 시 userRepository.save() 명시 저장
- Vacancy 알림 처리 시 waitlistRepository.save() 저장 처리
Copy link
Copy Markdown
Contributor

@3uomlkh 3uomlkh left a comment

Choose a reason for hiding this comment

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

save() 저장 확인했습니다!

Copy link
Copy Markdown
Contributor

@mannaKim mannaKim left a comment

Choose a reason for hiding this comment

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

수고하셨습니다! 👍

@pshshsh pshshsh merged commit d5a1eb6 into dev May 2, 2025
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.

[REFACTOR] Notification 관련 더티 체킹 제거 및 명시적 save 처리

3 participants