Skip to content

Conversation

@0Hooni
Copy link
Member

@0Hooni 0Hooni commented Oct 19, 2025

Summary by CodeRabbit

  • Bug Fixes

    • 개선된 이미지 배너 자동 스크롤 동작
  • Chores

    • 내부 프레임워크 의존성 추가
    • 네비게이션 로직 최적화

commit 4c23b03
Author: 0Hooni <thddudgns972@gmail.com>
Date:   Sun Oct 19 20:20:32 2025 +0900

    Squashed commit of the following:

    commit a9ec806
    Author: 0Hooni <thddudgns972@gmail.com>
    Date:   Mon Sep 15 09:48:09 2025 +0900

        chore/#177: 프레임워크 의존성 추가

    commit 3f356c8
    Author: 0Hooni <thddudgns972@gmail.com>
    Date:   Mon Sep 15 09:47:51 2025 +0900

        fix/#177: 배너 선택시 발생하던 인덱스 에러 해결

commit ae8a183
Author: 0Hooni <thddudgns972@gmail.com>
Date:   Fri Sep 12 20:45:39 2025 +0900

    docs/#176: 문제 범위 주석 추가

commit 3251ccb
Author: 0Hooni <thddudgns972@gmail.com>
Date:   Fri Sep 12 20:45:20 2025 +0900

    fix/#176: 홈화면 무한캐러셀 에러 임시 해결

    - 인덱스가 1 이상일때만 동작하도록 수정
@0Hooni 0Hooni self-assigned this Oct 19, 2025
@0Hooni 0Hooni added 🔄 refactor 프로덕션 코드 리팩토링, 파일 삭제, 네이밍 수정 및 폴더링 🐛 fix 버그 수정, 잔잔바리 수정, 병합 시 충돌 해결 labels Oct 19, 2025
@0Hooni 0Hooni linked an issue Oct 19, 2025 that may be closed by this pull request
@coderabbitai
Copy link

coderabbitai bot commented Oct 19, 2025

Caution

Review failed

The pull request is closed.

Walkthrough

LoginFeatureInterface.framework 의존성이 프로젝트에 추가되었으며, HomeReactor의 세부 정보 탐색 로직이 재구성되어 공유 헬퍼 함수로 통합되었습니다. 추가로 ImageBannerSectionCell의 가드 바인딩이 간소화되었습니다.

Changes

코호트 / 파일 변경 요약
프레임워크 의존성 추가
Poppool/PresentationLayer/Presentation/Presentation.xcodeproj/project.pbxproj
LoginFeatureInterface.framework이 PBXBuildFile, PBXFileReference로 추가되고 Frameworks 빌드 페이즈 및 프로젝트 구조에 연결됨
탐색 로직 리팩토링
Poppool/PresentationLayer/Presentation/Presentation/Scene/Home/Main/HomeReactor.swift
moveToDetail 헬퍼 함수를 도입하여 인라인 세부 정보 탐색 로직을 통합; 매개변수 타입을 Int에서 Int64로 변경; 인덱스 처리 조정 (indexPath.row == 0일 때 idList[0] 사용)
가드 바인딩 최적화
Poppool/PresentationLayer/Presentation/Presentation/Scene/Home/Main/View/ImageBannerSection/ImageBannerSection/ImageBannerSectionCell.swift
자동 스크롤 타이머 블록에서 guard let self = self를 guard let self로 단순화; 섹션 0에 항목이 2개 이상일 때만 스크롤 실행

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~20 minutes

프로젝트 설정 변경(framework 추가), 네비게이션 로직 리팩토링(타입 변경 및 헬퍼 함수 추가), UI 코드 최적화(가드 바인딩 단순화) 등 이질적인 변경이 여러 영역에 걸쳐 있으나, 각 변경은 명확한 패턴을 따릅니다.

Suggested reviewers

  • zzangzzangguy

Poem

🐰 새로운 LoginFeature 프레임워크가 도착했네,
탐색 로직은 깔끔하게 정리되고,
가드 바인딩은 쏜살같이 빨라졌어요!
구조도 타입도 한껏 다다다닥 개선되었다네,
이 멋진 변화를 함께 축하해요! 🎉

✨ Finishing touches
  • 📝 Generate docstrings
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch fix/#172-server-transfer-handle

📜 Recent review details

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between c73d9da and 4a98d49.

📒 Files selected for processing (3)
  • Poppool/PresentationLayer/Presentation/Presentation.xcodeproj/project.pbxproj (4 hunks)
  • Poppool/PresentationLayer/Presentation/Presentation/Scene/Home/Main/HomeReactor.swift (1 hunks)
  • Poppool/PresentationLayer/Presentation/Presentation/Scene/Home/Main/View/ImageBannerSection/ImageBannerSection/ImageBannerSectionCell.swift (1 hunks)

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 and usage tips.

@0Hooni 0Hooni merged commit f46faf2 into develop Oct 19, 2025
2 of 3 checks passed
@coderabbitai coderabbitai bot changed the title @coderabbitai Add LoginFeatureInterface and refactor HomeReactor logic Oct 19, 2025
@0Hooni 0Hooni deleted the fix/#172-server-transfer-handle branch October 19, 2025 11:26
@0Hooni 0Hooni changed the title Add LoginFeatureInterface and refactor HomeReactor logic [FEAT] 홈 화면 배너 에러 해결 Oct 29, 2025
0Hooni added a commit that referenced this pull request Nov 12, 2025
Squashed commit of the following:

commit 4c23b03
Author: 0Hooni <thddudgns972@gmail.com>
Date:   Sun Oct 19 20:20:32 2025 +0900

    Squashed commit of the following:

    commit a9ec806
    Author: 0Hooni <thddudgns972@gmail.com>
    Date:   Mon Sep 15 09:48:09 2025 +0900

        chore/#177: 프레임워크 의존성 추가

    commit 3f356c8
    Author: 0Hooni <thddudgns972@gmail.com>
    Date:   Mon Sep 15 09:47:51 2025 +0900

        fix/#177: 배너 선택시 발생하던 인덱스 에러 해결

commit ae8a183
Author: 0Hooni <thddudgns972@gmail.com>
Date:   Fri Sep 12 20:45:39 2025 +0900

    docs/#176: 문제 범위 주석 추가

commit 3251ccb
Author: 0Hooni <thddudgns972@gmail.com>
Date:   Fri Sep 12 20:45:20 2025 +0900

    fix/#176: 홈화면 무한캐러셀 에러 임시 해결

    - 인덱스가 1 이상일때만 동작하도록 수정
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

🐛 fix 버그 수정, 잔잔바리 수정, 병합 시 충돌 해결 🔄 refactor 프로덕션 코드 리팩토링, 파일 삭제, 네이밍 수정 및 폴더링

Projects

None yet

Development

Successfully merging this pull request may close these issues.

서버 이전 작업 대응

2 participants