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: 번쩍 상세 조회 GET API 구현 완료 #540

Merged
merged 12 commits into from
Jan 23, 2025
Merged

Conversation

hoonyworld
Copy link
Member

@hoonyworld hoonyworld commented Jan 22, 2025

👩‍💻 Contents

  • 번쩍 상세 조회 GET API를 구현했습니다.

📝 Review Note

모임 id? 번쩍 id?

  • 처음에 번쩍 상세 조회를 설계했었을 때, 번쩍 id로 조회할 수 있도록 하려고 했습니다.
  • 그런데 프론트 분들에게 물어본 결과 전체 모임 리스트 조회 시에도 모임 id를 사용하고 있다고 했었고, 번쩍 id를 프론트에서 새롭게 관리해야 한다는 부담감이 있을 것이라고 판단했습니다.
  • 따라서 프론트 분들은 meetingId만 알고 있고, 이를 조회 시 보내면 번쩍 엔티티에도 meetingId를 저장하기 때문에 이를 이용해서 번쩍 정보들을 조회하는 방식으로 구현하였습니다.
  • 따라서 저번에 번쩍 개설 시 주던 lightningId도 meetingId를 주도록 변경하였습니다.

userReader 객체를 통해 모임장 정보 조회 문의

MeetingCreatorDto meetingLeader = userReader.getMeetingLeader(lightning.getLeaderUserId());
  • 현재 LightningV2ServiceImpl에서도 모임장 정보를 조회하기 위해 해당 메서드를 사용하고 있습니다.
  • 이번 번쩍 상세 조회에서도 레디스의 캐시를 사용하게 되었는데, 혹시 getMeetingLeader를 2개의 서비스 레이어에서 접근하는 것에는 문제가 없는지 여쭙고 싶습니다. (문제는 없을 것 같은데 혹시 몰라서 여쭈어봤습니다!)

응답 response

image image
  • 스웨거로 테스트한 response 입니다.

📣 Related Issue

✅ 점검사항

  • docker-compose.yml 파일에 마이그레이션 한 API의 포워딩을 변경해줬나요?
  • Spring Secret 값을 수정하거나 추가했다면 Github Secret에서 수정을 해줬나요?
  • Nestjs Secret 값을 수정하거나 추가했다면 Docker-Compose.yml 파일 및 인스턴스 내부의 .env 파일을 수정했나요?

@hoonyworld hoonyworld added the 🎁 feature 새로운 기능 label Jan 22, 2025
@hoonyworld hoonyworld requested a review from mikekks January 22, 2025 15:30
@hoonyworld hoonyworld self-assigned this Jan 22, 2025
@hoonyworld hoonyworld linked an issue Jan 22, 2025 that may be closed by this pull request
1 task
Copy link

height bot commented Jan 22, 2025

Link Height tasks by mentioning a task ID in the pull request title or commit messages, or description and comments with the keyword link (e.g. "Link T-123").

💡Tip: You can also use "Close T-X" to automatically close a task when the pull request is merged.

Copy link
Member

@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.

고생하셨습니다!! 제가 큰 도움이 안돼 죄송합니다,,, 궁금한 부분은 질문드렸습니다 !!

@hoonyworld hoonyworld merged commit 4ed669f into develop Jan 23, 2025
1 check passed
@mikekks mikekks deleted the feat/#539 branch January 23, 2025 14:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🎁 feature 새로운 기능 size/L
Projects
None yet
Development

Successfully merging this pull request may close these issues.

feat: 번쩍 상세 조회 GET API 구현
2 participants