[FEAT/#122] AsyncImage로 변경

[chanmi1125]

Related issue 🛠

• closed [feat] 로딩 작업 #122

Work Description ✏️

• UrlImage의 SubcomposeAsyncImage → AsyncImage 교체
• 이미지 로딩 상태(isLoading) 처리

Uncompleted Tasks 😅

• 스켈레톤 적용

To Reviewers 📢

• 로딩 UI 디자인이 아직 정해지지 않아 일단 gray50 배경으로 처리했습니다. 해당 위치에 TODO 주석을 남겨 두었고, 디자인이 정해지면 그 분기만 수정하면 됩니다.
• 기존 실패 화면에서 modifier가 바깥과 안쪽에 두 번 적용되던 구조가 한 번만 적용되도록 바뀌었습니다. padding이 들어간 호출부는 실패 시 레이아웃이 기존과 조금 다를 수 있습니다.

Summary by CodeRabbit

• 개선 사항
  • 이미지 로딩 중이거나 로드에 실패하고 별도 플레이스홀더가 없는 경우 회색 배경을 표시합니다.
  • 플레이스홀더 이미지가 로딩 중, 이미지가 없을 때, 로드 실패 시 일관되게 표시됩니다.

[coderabbitai]

Note

Currently processing new changes in this PR. This may take a few minutes, please wait...

⚙️ Run configuration

• Configuration used: Repository: team-haphap/haphap-android/.coderabbit.yaml
• Review profile: CHILL
• Plan: Advanced
• Run ID: d5a3be02-0798-4428-8efb-716f5a41539e

📥 Commits

Reviewing files that changed from the base of the PR and between d869d0b and a024b64.

📒 Files selected for processing (1)

• app/src/main/java/com/haphap/app/core/designsystem/component/image/UrlImage.kt

 __________________________
< Needle. Haystack. Found. >
 --------------------------
  \
   \   (\__/)
       (•ㅅ•)
       / 　 づ

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

• Configuration used: Repository: team-haphap/haphap-android/.coderabbit.yaml
• Review profile: CHILL
• Plan: Advanced
• Run ID: 1fd9933b-9bea-4026-b566-c230a2a75cbd

📥 Commits

Reviewing files that changed from the base of the PR and between e6afe17 and d869d0b.

📒 Files selected for processing (1)

• app/src/main/java/com/haphap/app/core/designsystem/component/image/UrlImage.kt

Included review availability: This review used your included allowance. Your plan provides up to 1 included review per hour; 0 remain after this review.

---

📝 Walkthrough

Walkthrough

UrlImage이 SubcomposeAsyncImage 대신 AsyncImage를 사용합니다. URL별 로딩·오류 상태를 관리하고, placeholder, error, fallback painter를 지정합니다. 플레이스홀더가 없을 때 로딩 중이거나 오류가 발생하면 gray50 배경을 표시합니다.

Changes

UrlImage 로딩 처리

Layer / File(s)	Summary
AsyncImage 상태 및 플레이스홀더 처리 app/src/main/java/com/haphap/app/core/designsystem/component/image/UrlImage.kt	AsyncImage에 placeholder, error, fallback painter를 지정합니다. 콜백에서 URL별 로딩·오류 상태를 갱신합니다. 플레이스홀더가 없고 로딩 중이거나 오류 상태이면 gray50 배경을 적용합니다. 문서의 placeholderDrawable 설명도 수정합니다.

Priority: ⬇️ Low

Estimated code review effort: 2 (Simple) | ~10 minutes

Change: Feature

Suggested reviewers: vahkjsdf

Merge Risk: ⚪ Minimal · up to d869d

No actionable issue is established that should block merging. The reported loading placeholder remains a design follow-up.

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name	Status	Explanation	Resolution
Docstring Coverage	⚠️ Warning	Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%. Docstring coverage is scoped to functions touched by this diff. Analyzed 1 functions across 1 files.	Write docstrings for the functions missing them to satisfy the coverage threshold.

✅ Passed checks (4 passed)

Check name	Status	Explanation
Title check	✅ Passed	제목이 UrlImage의 SubcomposeAsyncImage를 AsyncImage로 변경하는 주요 내용을 명확하게 설명합니다.
Description check	✅ Passed	관련 이슈, 작업 내용, 미완료 작업, 리뷰어 안내를 포함합니다. 템플릿의 스크린샷 항목은 생략했지만 설명은 대부분 완전합니다.
Linked Issues check	✅ Passed	열린 직접 연결 이슈 #122의 제목은 로딩 작업이며, 구체적인 수용 기준은 없습니다. PR은 UrlImage를 AsyncImage로 바꾸고 onLoading·onSuccess·onError에서 로딩 및 오류 상태를 관리합니다. 플레이스홀더가 없을 때 로딩 또는 오류 배경을 표시합니다. 확인 가능한 이슈 요구사항을 충족합니다.
Out of Scope Changes check	✅ Passed	변경은 #122의 로딩 작업과 연결됩니다. AsyncImage 전환, 로딩·오류 상태 처리, 플레이스홀더 설명 수정은 해당 기능에 포함됩니다. 오류 시 modifier 적용 방식 변경도 같은 이미지 표시 경로의 변경이며, 무관한 변경은 확인되지 않았습니다.

• Fix all pre-merge checks with AI

✨ Finishing Touches

📝 Generate docstrings

• Commit to this branch
• Create a new PR

🧪 Generate unit tests (beta)

• Commit to this branch
• Create a new PR

• Autopilot · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands.}

[vahkjsdf]

굿굿띠니 수고했어용~

[vahkjsdf]

p1: 아무래도 이미지가 있어야하니까.. 이거 필수 파라미터로 옮겨야 할 듯,,ㅎㅎ

[Hiimynameiss]

수고많앗띠니~~ 어푸푸푸

[jiyoung2ee]

찬누 고생많았어용 ～～ 어푸푸👍