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

[TNT-115] 트레이너, 트레이니 연결 기능 구현 #65

Merged
merged 23 commits into from
Feb 7, 2025

Conversation

SeonJeongk
Copy link
Contributor

📝 작업 내용

연결 관련 API 연동 작업 진행했습니다

  • 트레이너 초대 코드 확인 API 연동
  • 트레이너 초대 코드 재발급 API 연동
  • 트레이니의 트레이너 초대 코드 인증 API 연동
  • 트레이니의 트레이너 연동 요청 API 연동
  • 트레이너의 연결된 트레이니 정보 불러오기 API 연동

전달사항

  • 기본 프로필 이미지가 .svg 형태로 넘어오고 있어 .png로 바꿔달라고 요청해둔 상태입니다.
    변경 이후 화면 다시 확인해보겠습니다.

  • 이전에 구두로 전달해주신 리뷰대로, 트레이니의 연결 플로우를 OnNextClick 버튼 하나로 관리할 수 있도록
    FormDataPT 수업 정보들과 프로필 정보를 저장하도록 수정했습니다.

  • 현재 trainerIdtraineeId를 넘겨주는 API가 없어서
    TrainerConnectCompleted, TraineeProfilePage화면에 기본 값이 보여지고 있습니다.
    API가 나오면 두 값 저장하는 로직 추가하겠습니다!

📸 실행 화면

트레이너

TrainerConnect.mp4

트레이니

TraineeConnect.mp4

🙆🏻 리뷰 요청 사항

연결된 트레이니 정보를 불러오는 API에 필요한 trainerIdtraineeId를 네비게이션을 통해 전달하려고 했습니다.
이 경우 해당 값을 사용하지 않는 CodeGenerationPage 화면으로 이동할 때 null 값이 들어가야 하는 문제가 있습니다.

그래서 현재 DataStore를 활용하여 값을 저장하고 불러오는 방식으로 구현했는데,
단순히 두 값만 저장하고 가져오는데 DataStore를 사용하는건 성능 저하를 유발 가능성이 있을 것 같습니다.

  • trainerId, traineeId가 필요 없는 화면이 있더라도 네비게이션을 통해 전달하는 것이 좋을까요?
  • 현재 DataStore를 사용하는 방식이 성능적으로 문제가 될 가능성이 있을까요?
  • 혹시 더 적절한 데이터 전달 방식이 있을까요..?!

의견 주시면 감사하겠습니다..! ;ㅅ;

👀 레퍼런스

@SeonJeongk SeonJeongk added ✨ Feat 기능 구현 🌻 선정 김씨 집안 막내 김선정 labels Feb 6, 2025
@SeonJeongk SeonJeongk added this to the 3차 스프린트 milestone Feb 6, 2025
@SeonJeongk SeonJeongk self-assigned this Feb 6, 2025
@github-actions github-actions bot requested a review from hoyahozz February 6, 2025 12:11
suspend fun regenerateInviteCode(): InviteCodeResponse

@GET("/trainers/invitation-code/verify/{code}")
suspend fun verifyInviteCode(
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
suspend fun verifyInviteCode(
suspend fun getVerifyInviteCode(

접두사로 HTTP 메소드 적어두면 한 눈에 알아볼 수 있을 것 같습니다~

@SeonJeongk SeonJeongk merged commit 1fcce19 into develop Feb 7, 2025
1 check passed
@SeonJeongk SeonJeongk deleted the feature/TNT-115 branch February 7, 2025 06:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
✨ Feat 기능 구현 🌻 선정 김씨 집안 막내 김선정
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[TNT-115] 파트너 연결 기능 구현
2 participants