Skip to content

fix: 타 유저 활동기록 더보기 목록이 조회되지 않음 - #952

Merged
Sadturtleman merged 1 commit into
developfrom
fix/951
Aug 29, 2026
Merged

fix: 타 유저 활동기록 더보기 목록이 조회되지 않음#952
Sadturtleman merged 1 commit into
developfrom
fix/951

Conversation

@Sadturtleman

@Sadturtleman Sadturtleman commented Aug 28, 2026

Copy link
Copy Markdown
Contributor

📌𝘐𝘴𝘴𝘶𝘦𝘴

📎𝘞𝘰𝘳𝘬 𝘋𝘦𝘴𝘤𝘳𝘪𝘱𝘵𝘪𝘰𝘯

  • 타 유저 페이지 > 활동 > 활동기록 더보기 진입 시 목록이 표시되지 않는 문제를 수정했습니다.
  • ActivityDetailActivity에서 타 유저 프로필을 직접 조회하도록 updateUserId() 호출을 추가했습니다.

원인

ActivityDetailActivityotherUserPageViewModelby viewModels()로 이 액티비티에 새로 생성되는 인스턴스라 OtherUserPageActivity의 것과 공유되지 않습니다. OtherUserPageViewModelupdateUserId()를 호출해야 프로필을 조회하는데 이 화면에서는 호출되지 않아 otherUserProfile이 계속 null이었습니다.

if (activities != null && userProfile != null) { ... }
else { activityDetailAdapter.submitList(emptyList()) }   // 프로필이 null이라 항상 여기로 빠짐

피드 조회 자체는 성공하지만 어댑터에 빈 리스트가 전달되어 화면에만 아무것도 그려지지 않았습니다.

내 활동은 MyPageViewModelinit에서 프로필을 조회하므로 영향이 없습니다.

📷𝘚𝘤𝘳𝘦𝘦𝘯𝘴𝘩𝘰𝘵

수정 전 수정 후
목록 항목 0개 (제목만 표시) 활동 목록 정상 표시

수정 전

GET /users/10032/feeds?lastFeedId=0&size=100  ->  200
{"feeds":[{"feedId":4553,"feedContent":"아아", ...}]}
화면 표시 텍스트: "활동"

수정 후

GET /users/10032/feeds?lastFeedId=0&size=100  ->  200
GET /users/profile/10032                      ->  추가됨
화면 표시 텍스트: "활동" / "test" / "4월 13일" / "아아" ...

💬𝘛𝘰 𝘙𝘦𝘷𝘪𝘦𝘸𝘦𝘳𝘴

Summary by CodeRabbit

  • 버그 수정
    • 다른 사용자의 활동에서 상세 화면을 열었을 때 해당 사용자 정보가 올바르게 전달되도록 개선했습니다.
    • 활동 상세 정보와 사용자 정보가 더욱 일관되게 갱신됩니다.

ActivityDetailActivity의 OtherUserPageViewModel은 OtherUserPageActivity와
다른 인스턴스여서 프로필이 조회되지 않았고, 프로필이 null이면 어댑터에
빈 리스트가 전달되어 목록이 비어 보였다.
@coderabbitai

coderabbitai Bot commented Aug 28, 2026

Copy link
Copy Markdown

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro Plus

Run ID: 1fae190f-d093-4c18-ae19-2f814a4b0dfb

📥 Commits

Reviewing files that changed from the base of the PR and between c0d1811 and 4e6f8ca.

📒 Files selected for processing (1)
  • app/src/main/java/com/into/websoso/ui/activityDetail/ActivityDetailActivity.kt

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


Walkthrough

타 사용자 활동 상세 화면이 타 사용자 프로필의 사용자 ID를 OtherUserPageViewModel에 전달합니다. 이를 통해 해당 화면에서 프로필 기반 활동 목록 조회가 수행됩니다.

Changes

타 사용자 활동 상세 조회

Layer / File(s) Summary
프로필 ViewModel 사용자 ID 연결
app/src/main/java/com/into/websoso/ui/activityDetail/ActivityDetailActivity.kt
SOURCE_OTHER_USER_ACTIVITY인 경우 otherUserPageViewModel.updateUserId(userId)를 호출합니다. 기존의 ActivityDetailViewModel 사용자 ID 설정과 활동 새로 고침은 유지됩니다.

Estimated code review effort: 1 (Trivial) | ~2 minutes

Merge Risk: ⚪ Minimal · up to 4e6f8

This change restores activity-list rendering for other users by loading the selected profile through the existing app flow. No actionable merge-blocking risk remains beyond normal checks and review.

Poem

토끼가 사용자 ID를 살짝 건네고
프로필 문이 활짝 열리네
활동 기록이 하나둘 나타나
빈 목록은 저 멀리 사라지고
당근처럼 데이터가 자라난다

🚥 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 2 functions across 1 files. Write docstrings for the functions missing them to satisfy the coverage threshold.
✅ Passed checks (4 passed)
Check name Status Explanation
Linked Issues check ✅ Passed [951]의 요구사항을 충족합니다. ActivityDetailActivity에서 타 유저 경로에 OtherUserPageViewModel의 사용자 ID를 전달하고 프로필 조회를 수행하게 하여, 프로필이 null이라 빈 목록이 표시되던 문제를 수정합니다. 내 활동 경로는 기존 동작을 유지합니다.
Out of Scope Changes check ✅ Passed 변경 사항은 ActivityDetailActivity의 타 유저 활동 상세 경로에서 사용자 ID를 전달하는 코드로 한정됩니다. 연결된 이슈 [951]의 범위를 벗어난 변경은 확인되지 않습니다.
Title check ✅ Passed 제목은 타 유저 활동기록 더보기 목록이 표시되지 않는 문제를 수정하는 변경 사항을 정확하고 간결하게 설명합니다.
Description check ✅ Passed 이슈, 작업 내용, 원인, 수정 내용, 검증 결과를 포함합니다. 스크린샷 섹션에는 실제 이미지 대신 수정 전후 동작과 API 결과가 작성되었고, 리뷰어 메모는 비어 있지만 비필수 항목이므로 설명은 충분합니다.
  • Fix all pre-merge checks with AI
✨ Finishing Touches 💡 1
📝 Generate docstrings 💡
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch fix/951

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

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

@devfeijoa devfeijoa left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

수고하셨습니다!

@Sadturtleman
Sadturtleman merged commit 04c5168 into develop Aug 29, 2026
3 checks passed
@Sadturtleman
Sadturtleman deleted the fix/951 branch August 29, 2026 05:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

fix: 타 유저 활동기록 더보기 목록이 조회되지 않음

2 participants