Skip to content

[Feature/#508] CMP ContributorAPI 구현 및 Contributor 목록에 연결#522

Merged
workspace merged 14 commits intodroidknights:2025/compose-multiplatformfrom
haeti-dev:feature/#508-contributor-api-connect
Jun 10, 2025
Merged

[Feature/#508] CMP ContributorAPI 구현 및 Contributor 목록에 연결#522
workspace merged 14 commits intodroidknights:2025/compose-multiplatformfrom
haeti-dev:feature/#508-contributor-api-connect

Conversation

@haeti-dev
Copy link
Copy Markdown
Contributor

@haeti-dev haeti-dev commented Jun 8, 2025

Issue

Overview (Required)

  • 2025/app 브렌치의 결과물과 똑같은 데이터를 보여주기 위해 같은 로직(Github API + 드나 연도 별 데이터 (~2024))을 사용하였습니다.
  • 기존 더미 데이터를 삭제하고, 위 데이터 소스를 사용하여 컨트리뷰터 목록을 보여줍니다.
  • KMP common에서는 JVM TreeMap을 사용하지 못하기 때문에 기존 toSortedMap() 메소드를 사용하지 못하여 entries.sortedByDescending() + List<Pair> 방식으로 대체하였습니다.
  • GitHub API를 사용해야 했기에 기존 DroidKnightsNetwork에서 githubHttpClient를 추가하였습니다.
  • 세션 모듈 쪽 상황과 비슷하게 SideEffect 처리는 시간이 부족하여 수행하지 못하였습니다.
  • 2025/app 브렌치의 결과물과 피그마의 디자인이 다르기에, 2025/app 버전과 싱크를 맞추었습니다.
  • KMP 작업이 처음이라 조금 어색한 부분이 있을 수 있습니다..!

Screenshot

_._api.mp4

Copy link
Copy Markdown
Member

@taehwandev taehwandev left a comment

Choose a reason for hiding this comment

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

고생하셨습니다.

Copy link
Copy Markdown
Contributor

@workspace workspace left a comment

Choose a reason for hiding this comment

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

@haeti-dev 짧은 시간이었는데, 수고하셨습니다!!

@workspace workspace merged commit 13bb4cc into droidknights:2025/compose-multiplatform Jun 10, 2025
9 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants