-
Notifications
You must be signed in to change notification settings - Fork 1
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
feat: 회원탈퇴, 로그아웃 시, 회원 위치 데이터 삭제 로직 추가 및 위치 데이터 유효기간 변경 #179
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
이거 너무 오버 핸들링 같은데요.. 트랜잭션에 묶이는 로직도 너무 많아지고 |
나중에 더 좋은 방법으로 해결해봅시당 |
Quality Gate passedIssues Measures |
choidongkuen
added a commit
that referenced
this pull request
Feb 11, 2024
* feat: 리뷰 등록시 회원 매너 온도 반영 기능 추가 및 리뷰 등록 리팩토링 (#158) * refactor: 회원 리뷰 등록 API 리팩토링 (#157) * feat: 회원 리뷰 등록시, 온도 반영 및 리뷰 피드백 반영 (#157) * feat: User 엔티티 메소드 추가 (#157) * feat: Review enum 필드 추가 (#157) * test: 리뷰 등록시 온도 업데이트에 대한 단위 테스트 (#157) * test: 리뷰 등록 기능 통합 테스트 (#157) * refactor: 회원 리뷰 등록 메소드 수정 (#157) * test: 불필요한 테스트 제거 및 CI 오류 수정 (#157) * test: MeetingRepository 테스트에서 시간과 id비교 비활서화 * refactor: 회원 탈퇴 URI 변경 (#162) * refactor: 회원 탈퇴 URI 수정 (#161) * test: 회원 탈퇴 URI 수정에 대한 테스트 수정 (#161) * fix: CI 에러 수정 (#161) * fix: 필드 값 비교를 위해 deprecated 된 메소드을 대체 (#161) * fix: 필드 값 비교를 위해 deprecated 된 메소드을 대체 (#161) * feat: fcm token 업데이트 api 추가 (#166) * fix: user_alert ddl의 pk에 auto_increment를 추가한다 * feat: token 업데이트 api를 추가한다 * fix: 모임 수정 시 이미지가 추가되는 버그 수정 (#169) * perf: upgrade GPT-3.5 to GPT-4 (#174) * refactor: 위치 기반 데이터 값 유효기간 변경 * refactor: 위치 기반 데이터 값 유효기간 변경 * refactor: 위치 기반 데이터 값 유효기간 변경 * feat: 모임 종료, 사용자 추천 알림을 추가 (#182) * feat: Alert Entity를 추가하고, 알림 전송 기능이 Entity를 파라미터로 받아 동작하게 수정한다 * refactor: MeetingAlerted를 BeforeMeetingAlerted로 직관적으로 변경한다 * feat: 모임 종료 알림을 추가한다 * feat: 유저가 추천받았을때 알림 전송 기능을 추가한다 * feat: 유저가 한달 동안 받은 알람을 조회하는 기능 개발 (#185) * fix: flyway schema에 _이 하나밖에 없는 버그를 수정한다 * feat: 유저가 한달동안 받은 알람을 조회하는 기능을 개발한다 * refactor: AlertRepository의 finalAll 메소드를 findAllByCreatedAt으로 변경한다 * feat: 북마크 추가, 취소 기능 구현 (#184) * chore: 북마크 스키마 추가 * feat: 북마크 추가, 삭제 기능 * test: 북마크 추가, 삭제 테스트 작성 - 북마크 추가 케이스 - 북마크 취소 케이스 - 모임 단일 응답에 북마크 여부 추 * feat: bookmarked -> isBookmarked, V1__ 언더바 누락 수정 * fix: flyway_schema alert userId를 user_id로 변경한다 * fix: flyway 13 의 alert table문을 변경한다 * refactor: 알람 조회에 읽은알람인지, 언제 발행되었는지를 추가 (#189) * feat: 회원탈퇴, 로그아웃 시, 회원 위치 데이터 삭제 로직 추가 및 위치 데이터 유효기간 변경 (#179) * refactor: 주변 위치 데이터 응답 데이터 구조 변경 및 유효기간 30 초 설정 (#178) * feat: 회원탈퇴, 로그아웃 시 저장되어 있는 위치 정보 Redis 에서 삭제 (#178) * feat: 회원탈퇴, 로그아웃 시 저장되어 있는 위치 정보 Redis 삭제 메소드 추가 (#178) * test: 로직 수정에 따른 단위 테스트 수정 (#178) * refactor: UserService 로그아웃, 회원 탈퇴 시 위치 데이터 삭제 로직 제거 (#178) * refactor: logout @transactional 제거 (#178) * refactor: TeumTeumService 리팩토링 (#178) * test: 회원탈퇴, 로그아웃 유닛 테스트 수정 (#178) * feat: 모임 참여자 정보 조회 기능을 구현 (#192) * feat: 모임 참여자 정보 조회 API 기능 구현 (#178) * test: 테스트 코드 구현 및 테스트 (#178) --------- Co-authored-by: devxb <[email protected]> Co-authored-by: xb205 <[email protected]> Co-authored-by: ddingmin <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
🚀 어떤 기능을 개발했나요?
🕶️ 어떻게 해결했나요?
1분
->30초
로 변경🦀 이슈 넘버