Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[CVW-024] SettingCell 수정 #40

Merged
merged 5 commits into from
Feb 2, 2025
Merged

[CVW-024] SettingCell 수정 #40

merged 5 commits into from
Feb 2, 2025

Conversation

GomHyeok
Copy link
Collaborator

@GomHyeok GomHyeok commented Jan 31, 2025

구현내용

ㅇ 1. SettingCellViewModel 기능 삭제

  • 해당 기능을 SettingViewModel에서 동작하도록 변경하였습니다.
  • SettingCell에 필요한 정보는 SettingRO를 생성하여 동작하도록 하였습니다.

ㅇ 2. I18N Stream 의존성 삭제

  • I18NStream에 대하여 구독하고 있던 것을 삭제하였습니다.
  • 해당 부분에 대하여 기존에는 I18NStream을 구독하는 것이 맞다고 생각하였으나, Setting값이 변경되는 것을 이미 계속해서 확인하고 있기 때문에 I18NStream이 불필요하다고 판단되었습니다.

고민사항

ㅇ 1. CreateRO()에서 state를 인자로 받는 상태

  • state의 변경에 따라 새로운 RO 생성이 필요하다고 생각되어 state값을 인자로 받고 있지만 해당 코드가 옳은 코드라는 확신이 없습니다.

기타(이미지 포함 기타 설명)

Simulator.Screen.Recording.-.iPhone.16.Pro.-.2025-01-31.at.10.05.12.mp4

- 기존에 필요한 ViewModel에 필요한 정보 가지도록 변경
- 각 toggle Event는 콜백 호출 하도록 변경 예정
- 다음 작업은 SettingCell에서 RO이용해서 View 랜더링 하도록 변경
- 정상적으로 toggle 작동하는 것 확인 완료
- toggle작동시 action에 이벤트 처리하는 부분 코드 수정 필요
- Setting값 변경에 따른 RO 변경 반영되도록 변경
@GomHyeok GomHyeok requested a review from J0onYEong January 31, 2025 01:05
@GomHyeok GomHyeok merged commit 6379591 into main Feb 2, 2025
1 check passed
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.

1 participant