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: APP BE 와의 통신을 위한 모임 목록 조회 API 개발 #486

Merged
merged 16 commits into from
Nov 17, 2024

Conversation

mikekks
Copy link
Member

@mikekks mikekks commented Nov 15, 2024

👩‍💻 Contents

  • APP BE 와의 통신을 위한 모임 목록 조회 API 개발 했습니다.
  • 아직 모임 목록 조회 정책이 확실히 정해지지 않았지만 APP 쪽에서 딜레이가 계속 생길 것 같아 먼저 구현했습니다.
스크린샷 2024-11-15 오후 6 07 09

📝 Review Note

  • 저희가 외부 서비스를 사용하는 경우 external, 외부 서비스가 저희 서비스를 사용하는 경우 internal 이라고 정의하고자 했습니다. 의견 궁금합니다!
  • 기존에는 플그 토큰 받아서 진행하고자 했습니다. 하지만 플그와의 소통을 줄이고 싶어 화이트 리스트에 등록해서 구현했습니다. 의견 궁금합니다! 현재는 단순히 플그 id만 받아서 구현한 상황입니다.

📣 Related Issue

✅ 점검사항

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

@mikekks mikekks added the 🎁 feature 새로운 기능 label Nov 15, 2024
@mikekks mikekks self-assigned this Nov 15, 2024
Copy link

height bot commented Nov 15, 2024

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

@hoonyworld hoonyworld left a comment

Choose a reason for hiding this comment

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

고생하셨습니다 민규님!
페이지네이션 로직을 살펴본 결과 페이지 네이션 구현 로직에는 문제가 없어보이나 requestBody가 아닌 requestParam으로 GET 요청 방식을 변경하여야 할 것 같습니다!

저희가 외부 서비스를 사용하는 경우 external, 외부 서비스가 저희 서비스를 사용하는 경우 internal 이라고 정의하고자 했습니다. 의견 궁금합니다!

  • 답변 : 매우 좋은 것 같습니다!

기존에는 플그 토큰 받아서 진행하고자 했습니다. 하지만 플그와의 소통을 줄이고 싶어 화이트 리스트에 등록해서 구현했습니다. 의견 궁금합니다! 현재는 단순히 플그 id만 받아서 구현한 상황입니다.

  • 답변 : 이 부분은 잘 이해가 안되었는데, 자세히 설명해주실 수 있으실까요? 플그 id만 받아서 구현한다는 것은 차단여부를 판단하기 위해 플그 MemberBlock 테이블에 접근해서 orgId를 가져온다는 걸까요?

@mikekks
Copy link
Member Author

mikekks commented Nov 16, 2024

고생하셨습니다 민규님! 페이지네이션 로직을 살펴본 결과 페이지 네이션 구현 로직에는 문제가 없어보이나 requestBody가 아닌 requestParam으로 GET 요청 방식을 변경하여야 할 것 같습니다!

기존에는 플그 토큰 받아서 진행하고자 했습니다. 하지만 플그와의 소통을 줄이고 싶어 화이트 리스트에 등록해서 구현했습니다. 의견 궁금합니다! 현재는 단순히 플그 id만 받아서 구현한 상황입니다.

  • 답변 : 이 부분은 잘 이해가 안되었는데, 자세히 설명해주실 수 있으실까요? 플그 id만 받아서 구현한다는 것은 차단여부를 판단하기 위해 플그 MemberBlock 테이블에 접근해서 orgId를 가져온다는 걸까요?

넵 맞습니다 !! 아래 과정 자세히 설명했습니다!

  • 과정
  1. APP BE 로 부터 플그 id(orgId)를 requestParam으로 받아온다.
  2. 받은 플그 id로 MemberBlock 테이블에 접근해서 차단한 유저의 orgId를 가져온다.
  3. 필터링하여 모임 목록을 반환한다.

Copy link
Member

@hoonyworld hoonyworld left a comment

Choose a reason for hiding this comment

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

넵 맞습니다 !! 아래 과정 자세히 설명했습니다!

  • 과정
  1. APP BE 로 부터 플그 id(orgId)를 requestParam으로 받아온다.
  2. 받은 플그 id로 MemberBlock 테이블에 접근해서 차단한 유저의 orgId를 가져온다.
  3. 필터링하여 모임 목록을 반환한다.

이해했습니다 😊
차단 여부를 이미 플그 DB의 MemberBlock으로 부터 가져오고 있기 때문에, 해당 방법 좋은 것 같습니다!

Copy link
Member

@hoonyworld hoonyworld left a comment

Choose a reason for hiding this comment

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

LGTM~

@mikekks mikekks merged commit 6542937 into develop Nov 17, 2024
1 check passed
@mikekks mikekks deleted the feat/#484 branch November 17, 2024 07:01
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: app be를 위한 전체 모임 조회 API 개발
2 participants