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.

Sign up for GitHub

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

Jump to bottom

[TNT-213] 트레이너 회원목록 페이지 작성 #62

Merged
merged 4 commits into from
Feb 10, 2025
Merged

[TNT-213] 트레이너 회원목록 페이지 작성 #62

merged 4 commits into from
Feb 10, 2025

Conversation

syss220211
Copy link
Member

📌 What is the PR?

  • 트레이너 > 마이페이지 > 회원목록 화면을 작성하였습니다.

🪄 Changes

  • 회원 목록 화면 작성
  • DTO, UseCase 작성
  • 우선 API가 다르긴한데.. 혹시 모를거 같아서 냅뒀습니다.. 어떻게 할까욥?
  • 화면은 데이터가 없어서 마저 마무리는 안 했어요(Cell 만 만들었구, 추후에 API 연결후에 Foreach로 마무리 해놓을 예정입니다.)

🙆🏻 To Reviewers

  • API를 어디서 쓰긴 쓸텐데.. 어디인지 모르겠습니다 (?) 혹시 아시면 말씀부탁드려요! 제가 그쪽으로 옮겨 놓을게요..

💭 Related Issues

@syss220211
[Feat] 트레이너 회원 목록 usecase, dto 정의
06959be 
- 화면 작성 (네비게이션 추가)
- 회원이 없는 경우의 EmptyView 화면 작성
@syss220211
[Feat] 트레이너 마이페이지 회원목록 화면 작성
f14e28c
@syss220211 syss220211 added the ✨Feat 새로운 기능 구현 (새로운 로직 추가, UI 구현 등) label Feb 8, 2025
@syss220211 syss220211 requested a review from FpRaArNkK February 8, 2025 19:13
@syss220211 syss220211 self-assigned this Feb 8, 2025
FpRaArNkK
FpRaArNkK requested changes Feb 8, 2025
View reviewed changes
Copy link
Contributor

@FpRaArNkK FpRaArNkK left a comment •
edited
Loading

Choose a reason for hiding this comment

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

작성해주신 getMembersList에 해당하는 API는 각 유저 마이페이지용 API에요!
트레이너 마이페이지 조회 화면에서 사용해주시면 될 것 같고, 추후 회원목록 API 스펙 나오면 추가하는 걸로 합시당
API 관련 구조 변경 작업은 시간 문제로 당장 진행하기는 힘들것 같아요 ㅠㅠ
마감한 후에 추후 리팩토링을 노려보는걸로 하시죠! 지금은 extension mapper 만으로 충분할 것 같습니당

TnT/Projects/Domain/Sources/Policy/Extension/Date+.swift Outdated
Copy link
Contributor

Choose a reason for hiding this comment

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

위치 변경에 이유가 있을까요??

Copy link
Member Author

Choose a reason for hiding this comment

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

음 해당 부분 제가 변경한 기억이 없습니다만.. 기존에 Domain > Sources > Policy > Extension > Date+ 가 맞을지요? 맞다면 리베이스하면서 같이 올라간것이지 않을까.. 생각합니다.

Copy link
Contributor

Choose a reason for hiding this comment

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

음... 저도 변경하지 않았어서, 기존 Develop과 다른 수정된 사항만 PR에 보일거에요!
일단 저희가 의도한 방향은 Domain/Sources/Extension 이 맞으니 위치 변경해주시면 좋을 것 같습니다 :)

Copy link
Member Author

Choose a reason for hiding this comment

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

넵

FpRaArNkK
FpRaArNkK reviewed Feb 9, 2025
View reviewed changes
TnT/Projects/DIContainer/Sources/DIContainer.swift Outdated
@@ -30,6 +30,10 @@ private enum SocialUseCaseKey: DependencyKey {
static let liveValue: SocialLoginUseCase = SocialLoginUseCase(socialLoginRepository: SocialLogInRepositoryImpl(loginManager: SNSLoginManager()))
}

private enum TrainerUseCaseKey: DependencyKey {
Copy link
Contributor

Choose a reason for hiding this comment

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

UseCaseRepoKey가 맞는 것 같아요! 추후 TrainerUseCase 등록을 위해 바꿔놓읍시당

Copy link
Member Author

Choose a reason for hiding this comment

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

넵 반영했습니다.

@FpRaArNkK FpRaArNkK mentioned this pull request Feb 9, 2025
Merged
@FpRaArNkK FpRaArNkK merged commit 78b0919 into develop Feb 10, 2025
@FpRaArNkK FpRaArNkK deleted the TNT-213-TrainerTraineeList branch February 12, 2025 08:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@FpRaArNkK FpRaArNkK FpRaArNkK approved these changes

Assignees

@syss220211 syss220211

Labels
✨Feat 새로운 기능 구현 (새로운 로직 추가, UI 구현 등)
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

[TNT-213] 트레이너 회원 목록 작성
2 participants
@syss220211 @FpRaArNkK