Skip to content

Comments

[Chore] 리팩토링에 사용된 중복된 코드 제거 #30

Merged
oungsi2000 merged 1 commit intodevelopfrom
chore/29
Jan 19, 2026
Merged

[Chore] 리팩토링에 사용된 중복된 코드 제거 #30
oungsi2000 merged 1 commit intodevelopfrom
chore/29

Conversation

@oungsi2000
Copy link
Contributor

#️⃣ 이슈 번호

#29


🛠️ 작업 내용

  • UI에서 Flow의 이벤트를 구독하는 데 사용되던 ObserveAsEvents 컴포저블의 위치를 변경하고, 더 이상 사용되지 않는 테스트용 Flow 확장 함수 파일을 삭제했습니다.

  • ObserveAsEvents.kt 경로 변경:

    • presentation/common/component/ 디렉터리에 있던 ObserveAsEvents.kt 파일을 삭제하고, presentation/common/으로 이동된 파일을 사용하도록 SettingFragment의 import 구문을 수정했습니다.
  • FlowExtension.kt 삭제:

    • 테스트 코드에서 Flow 이벤트를 관찰하기 위해 사용되던 observeEventobserveMultipleEvent 유틸리티 함수가 포함된 FlowExtension.kt 파일을 삭제했습니다.

🙇🏻 중점 리뷰 요청


📸 이미지 첨부 (Optional)

UI에서 Flow의 이벤트를 구독하는 데 사용되던 `ObserveAsEvents` 컴포저블의 위치를 변경하고, 더 이상 사용되지 않는 테스트용 Flow 확장 함수 파일을 삭제했습니다.

- **`ObserveAsEvents.kt` 경로 변경:**
    - `presentation/common/component/` 디렉터리에 있던 `ObserveAsEvents.kt` 파일을 삭제하고, `presentation/common/`으로 이동된 파일을 사용하도록 `SettingFragment`의 import 구문을 수정했습니다.

- **`FlowExtension.kt` 삭제:**
    - 테스트 코드에서 Flow 이벤트를 관찰하기 위해 사용되던 `observeEvent`와 `observeMultipleEvent` 유틸리티 함수가 포함된 `FlowExtension.kt` 파일을 삭제했습니다.
@oungsi2000 oungsi2000 self-assigned this Jan 14, 2026
@coderabbitai
Copy link

coderabbitai bot commented Jan 14, 2026

Important

Review skipped

Auto reviews are disabled on base/target branches other than the default branch.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.


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.

Copy link
Contributor

@parkjiminnnn parkjiminnnn left a comment

Choose a reason for hiding this comment

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

고생요~

@oungsi2000 oungsi2000 merged commit f6df42a into develop Jan 19, 2026
8 checks passed
@oungsi2000 oungsi2000 deleted the chore/29 branch January 19, 2026 03:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants