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

[Refactor] #237 - DetailClip, EditClip MVVM 리팩토링 #238

Open
wants to merge 5 commits into
base: develop
Choose a base branch
from

Conversation

mini-min
Copy link
Member

@mini-min mini-min commented Dec 19, 2024

✨ 해결한 이슈

🛠️ 작업내용

코드가 역시 조금 복잡하고 깁니다... 예상 못한 이슈가 있을 수 있으니 지적이나 질문 사항있으면 얼마든지 환영하고 받겠습니다 ^__^

1. DetailClipViewModel Combine 사용 리팩토링

  • Input : 링크 리스트 요청 (true- 전체,, false-클립), 세그멘트 수정, 링크 타이틀 수정, 링크 클립 이동 클릭 시, 링크 클립 선택 시, 링크 이동 최종 버튼 클릭 시, 링크 삭제
  • Output : 리로드, 이름 변경, 클립 데이터 리로드, 버튼 활성화, 클립 이동 결과, 링크 삭제 결과

2. DetailClipViewModel + ChangeClipViewModel 병합

  • 민재 작업 담당 -> DetailClipViewModel (링크 리스트 관련 처리 뷰모델)
  • 준혁 3차 스프린트 담당 -> ChangeClipViewModel (링크 카테고리 변경 관련 처리 뷰모델)

위 두 뷰 모델이 각자의 작업으로 분리되어 있어, 하나의 DetailClipViewModel로 합치는 작업도 수행했습니다.

3. EditClipViewModel Combine 사용 리팩토링

  • Input : 클립 리스트 요청, 클립 삭제 버튼 클릭, 클립 이름 텍스트 필드 입력, 클립 수정 버튼 클릭, 링크 순서 변경
  • Output : 리로드, 클립 삭제 결과, 클립 이름 중복 체크, 클립 순서 결과

4. 기타 버그 수정 및 코드 정리

  • 클립 이름 수정 관련 -> 중복 체크에 따른 바텀시트 높이 조정 코드는 불필요해서 삭제했습니다.
  • EditLinkBottomSheetView - clearButton 클릭 시 완료 버튼이 비활성화되도록 수정했습니다. (isButtonClicked = false)
  • 가독성 떨어지는 일부 코드 줄바꿈 처리 수정했습니다.

✅ Checklist

  • 필요없는 주석, 프린트문 제거했는지 확인
  • 컨벤션 지켰는지 확인

@mini-min mini-min added ♻️ Refactor 전면 수정이 있을 때 사용합니다 🐻‍❄️ 민재 민재 선생님 작업 labels Dec 19, 2024
@mini-min mini-min self-assigned this Dec 19, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
♻️ Refactor 전면 수정이 있을 때 사용합니다 🐻‍❄️ 민재 민재 선생님 작업
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Refactor] DetailClip, EditClip ViewModel Combine 적용
1 participant