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: admin 모듈 개발 및 회원 목록 조회 구현 #266

Merged
merged 13 commits into from
Dec 2, 2024

Conversation

mikekks
Copy link
Collaborator

@mikekks mikekks commented Nov 30, 2024

📝 PR 타입

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

📢 변경 사항

  • admin 모듈을 추가했습니다.

❗️To Reviewer

  • admin 모듈을 domain 과 완전히 분리하고 싶었는데 사실 아직 admin 기능이 매우 작기도 하고, 확장 가능성이 엄청크진 않을 것 같다고 생각했습니다. 그래서 일단 domain 모듈내에 repository로 분리하여 구현했습니다. 그리 좋은 구조는 아니지만 나쁘지 않을 것 같은데 의견 궁금합니다!
  • 어드민 모듈쪽에 자체 로그인이나 필터로 인가를 확인하면 좋을 것 같은데 이건 시간 이슈로 일단 담백하게 구현했습니다,,
  • 저희 멤버가 몇백명이기에 페이지네이션이 필요하다고 생각했습니다! 그래서 그 기능도 일단 최대한 간단히 구현했씁니다,,,

@clean2001 : recentActivityDate 값을 null 로 일단 박아뒀습니다! 구현하실 때 요 부분만 수정하시면 될 것 같습니다!

  • 🚨 현재 어드민 컨테이너는 {기존 domain}/admin 일 경우 리버스프록시 해주고 있습니다! 새로 도메인을 만드는게 좋을까요?! 일단 dev에는 ci-cd 수정해서 반영했습니다!

⚙️ 테스트 결과

스크린샷 2024-12-01 오전 1 32 53

발생한 쿼리 첨부

👉 반영 브랜치

  • feat/
  • closed #

@mikekks mikekks added the 🚀 feature 새로운 기능 개발 label Nov 30, 2024
@mikekks mikekks self-assigned this Nov 30, 2024
Copy link
Collaborator

@clean2001 clean2001 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 merged commit e912fb0 into develop Dec 2, 2024
3 checks passed
@mikekks mikekks deleted the feat/LA-20 branch December 2, 2024 03:56
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