Hotfix notification bugs - Githubissues

peer-42seoul / Peer-Backend

Peer service backend repository

2 stars 4 forks source link

Hotfix notification bugs #869

Closed Paul2021-R closed 9 months ago

Paul2021-R commented 9 months ago

변경 사항

주요 문제 포인트.

팀 쪽 권한 및 상태에 따라 알림이 전달되어야 하는데, 그렇게 되지 못한 점 발견
쇼케이스 쪽 DTO 잘못되어 있으나, 그것을 수정하지 않은체 진행한 것 발견함.팀ID가 전달되지 않아, 대상이 비정상적으로 되어 알림 이벤트가 뜨지 않는 문제 발견
멘트가 중의적이거나 일관적이지 않은 문제들

주요 해결 리스트

알림 쪽 DB에 필요한 값들 수정하는 JPQL에 @Modifying 어노테이션 추가
쪽지 전달 관련 알림 전체 수정
회원가입 시 나오는 문구 수정
팀 전반 알림 위치, 알림 관련 로직 수정 / 팀원 탐색 메소드에 대해 팀 상태 조건 추가
모집글, 쇼케이스 쪽 관련 댓글, 좋아요 등에서 로직에 맞춰 알림이 갈 수 있도록 수정함.

테스트 결과

전체 테스트 사이트를 통해 직접 진행하여 작동여부 파악 완료

sinryuji commented 9 months ago

🎉 @wochae 님 랜덤 리뷰어로 당첨되셨습니다! 리뷰를 부탁드립니다. 🙏

sonarcloud[bot] commented 9 months ago

Quality Gate passed

Issues
5 New issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarCloud