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

[SAMBAD-274] 마지막 접속 모임 정보 갱신 로직 구현 #123

Merged
merged 2 commits into from
Aug 22, 2024

Conversation

kkjsw17
Copy link
Collaborator

@kkjsw17 kkjsw17 commented Aug 22, 2024

✔️ PR 타입(하나 이상의 PR 타입을 선택해주세요)

  • 기능 추가
  • 버그 수정
  • 의존성, 환경 변수, 빌드 관련 코드 업데이트
  • 기타 사소한 수정

📝 개요

  • user.last_meeting_id 칼럼을 추가합니다.
    • null일 경우, 모임 목록의 첫 번째 모임 id를 제공합니다.
  • PATCH /v1/users/last-meeting API를 추가합니다.
  • 기존 GET /v1/meetings API에 마지막 접속 모임 필드를 추가합니다.

🔗 ISSUE 링크

@kkjsw17 kkjsw17 self-assigned this Aug 22, 2024
Copy link

github-actions bot commented Aug 22, 2024

Unit Test Results

4 tests   4 ✔️  0s ⏱️
1 suites  0 💤
1 files    0

Results for commit b481d1a.

♻️ This comment has been updated with latest results.

Copy link
Member

@LeeHanEum LeeHanEum left a comment

Choose a reason for hiding this comment

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

수고하셨습니다!

@ApiResponse(responseCode = "403", description = "USER_NOT_MEMBER_OF_MEETING")
})
@PatchMapping("/last-meeting")
public ResponseEntity<Void> updateLastMeeting(@UserId Long userId, @RequestBody LastMeetingRequest request) {
Copy link
Member

Choose a reason for hiding this comment

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

개행 확인 부탁드립니다!

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

개행 혹시 어떤 부분이 잘못되었나요??

@kkjsw17 kkjsw17 merged commit 2bb3ec1 into develop Aug 22, 2024
2 checks 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.

2 participants