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

feat: 최근 접속 시간 조회 기능 구현 #267

Merged
merged 10 commits into from
Dec 11, 2024
Merged

Conversation

clean2001
Copy link
Collaborator

📝 PR 타입

  • 기능 추가
  • 기능 수정
  • 기능 삭제
  • 리팩토링
  • docs 작업, swagger 작업
  • 의존성, 환경 변수, 빌드 관련 코드 업데이트

📢 변경 사항

api 모듈

  • aop로 요청이 들어올 때마다 유저의 이용시간을 redis에 기록하는 코드를 추가했습니다. (prod: redis 1번 db / dev & local: redis 2번 db)

admin 모듈

  • 기존 회원 정보 조회 API 응답 객체에서 null이었던 recentActivityDate를 레디스에 저장된 값으로 수정했습니다.
  • 최근 6개월 이상 접속 기록이 없을 시, 값이 null입니다

❗️To Reviewer

⚙️ 테스트 결과

4번 유저로 요청 보내고 레디스의 2번 db에 저장된 값
스크린샷 2024-12-02 오후 8 12 08

api 응답 값
스크린샷 2024-12-02 오후 9 03 05

발생한 쿼리 첨부

👉 반영 브랜치

  • feat/LA-27-2
  • closed #

@clean2001 clean2001 added the 🚀 feature 새로운 기능 개발 label Dec 2, 2024
@clean2001 clean2001 self-assigned this Dec 2, 2024
Copy link
Collaborator

@mikekks mikekks left a comment

Choose a reason for hiding this comment

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

고생하셨습니다 ~! 궁금한 부분은 리뷰 남겼습니다 !! :)

@mikekks mikekks self-requested a review December 11, 2024 11:27
Copy link
Collaborator

@mikekks mikekks left a comment

Choose a reason for hiding this comment

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

고생많으셨습니다 !!

@clean2001 clean2001 merged commit d8fd787 into develop Dec 11, 2024
3 checks passed
@clean2001 clean2001 deleted the feat/LA-27-2 branch December 11, 2024 14:32
clean2001 added a commit that referenced this pull request Dec 11, 2024
* feat: 최근 접속 시간 조회 기능 구현

* chore: GetMemberActivityResponse에서 변수명 수정

* chore: RedisConfig 통합 및 objectMapper 관련 변경

* chore: RecentActivityDto 생성 및 적용

* chore: dev, prod에 redis 컨테이너 추가

* chore: infra > development > redis.conf 추가

* fix: dev > docker-compose.yaml volume 정보

* fix: redis에도 network 정보 추가

* fix: dev > nginx.conf 수정

---------

Co-authored-by: mikekks <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🚀 feature 새로운 기능 개발
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants