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/#76] 소개 페이지 구현 #91

Merged
merged 11 commits into from
Jan 25, 2024
Merged

[FEAT/#76] 소개 페이지 구현 #91

merged 11 commits into from
Jan 25, 2024

Conversation

unam98
Copy link
Collaborator

@unam98 unam98 commented Jan 25, 2024

📌 개요

✨ 작업 내용

  • characterId 값을 대응되는 drawable로 변환해주는 IdMapper 유틸 클래스 추가
  • AuthUtils 클래스 추가
  • 주변 탐색 API 연결 (테스트 완료)
  • 소개 API 연결 (테스트 완료)
  • 흔들기 3초 시 자동으로 TopicActivity로 페이지 전환
  • 흔들기 뷰 탐색된 유저의 characterId에 대응되는 색깔로 세팅 (다크/라이트 확인 필요)
  • 탐색된 유저들의 관심사를 동적으로 뷰 추가

✨ PR 포인트

  • Familiar Fragment 히스토리(백스택) 관리 안 돼있는 것 인지하고 있습니다. naviGraph를 써본 적이 없어서 우선 냅뒀습니다.

  • post API의 경우 필요한 requestBody를 만들어서 repository에서 인자로 넣어주는데 지금 저희의 멀티모듈 환경에선 domain이 data를 참조할 수가 없어서 제가 싱글모듈로 해오던 방법이 안 됩니다. 이 부분을 뒤늦게 인지했으나 시간적 여유가 없어 코드 퀄리티는 버리고 우선 돌아가게만 해놓았습니다. 이러한 이슈 때문인지 domain 모듈이 좀 혼잡스럽게 섞이고 있는 느낌인데 이 모듈의 역할과 관련하여 추후 의견 나눠보고싶습니다.

  • api에서 내려주는 속성값들 이름이 알아보기 어려워서 시간 관계상 qa 때 잡으려고 일단 적절해보이는 아무 속성이나 바인딩해놓았습니다.

남은 작업

  • Topic API 연결 및 UI 바인딩
  • Familiar Fragment 히스토리 관리
  • 블러 효과 구현
  • 후면 카드 작업 보충
  • 뒤집기 애니메이션 추가

📸 스크린샷/동영상

@unam98 unam98 added 우남 FEAT 새로운 기능 구현 labels Jan 25, 2024
@unam98 unam98 self-assigned this Jan 25, 2024
Copy link
Collaborator

@kkk5474096 kkk5474096 left a comment

Choose a reason for hiding this comment

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

domain은 순수하게 kotlin으로 작성되어야 하며 다른 라이브러리나 모듈들의 의존성 없이 존재해야합니다. 그러므로 data를 참조하는 방식도 멀티모듈을 사용할 때 뿐만 아니라 싱글 모듈에서도 참조해서는 안되며 클린 아키텍처를 위배할 수 있습니다.

@unam98 unam98 merged commit e2c34d4 into main Jan 25, 2024
1 check passed
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.

[FEAT] 소개서 확인
2 participants