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] #89 ViewModel 코드 리펙토링 #90

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

Conversation

l5x5l
Copy link
Contributor

@l5x5l l5x5l commented Jan 19, 2025

Key Changes

아래 화면의 viewModel 들을 인터페이스-구현체를 사용하는 방식으로 수정했습니다.

  • 링크 추가/수정 화면
  • 포킷 추가/수정 화면
  • 알람 목록 화면
  • 링크 목록 화면 (안읽음 목록, 즐겨찾기 목록)
  • 검색 화면
  • 포킷 상세 화면
    Resolves: [REFACTOR] 코드 리펙토링 #89

PR 유형

어떤 변경 사항이 있나요?

  • 새로운 기능 추가
  • 버그 수정
  • CSS 등 사용자 UI 디자인 변경
  • 코드에 영향을 주지 않는 변경사항(오타 수정, 탭 사이즈 변경, 변수명 변경)
  • 코드 리팩토링
  • 주석 추가 및 수정
  • 문서 수정
  • 테스트 추가, 테스트 리팩토링
  • 빌드 부분 혹은 패키지 매니저 수정
  • 파일 혹은 폴더명 수정
  • 파일 혹은 폴더 삭제

To Reviewers

  • bottomSheet 관련된 부분은 아예 별도의 브랜치로 빼서 작업하는게 더 좋을 것 같아 viewModel 인터페이스로 분리한 부분만 pr 보냅니다!

PR Checklist

PR이 다음 요구 사항을 충족하는지 확인하세요.

  • 커밋 메시지 컨벤션에 맞게 작성했습니다.
  • 정해진 코딩 컨벤션에 맞게 작성했습니다.
  • 변경 사항에 대한 테스트를 했습니다.(버그 수정/기능에 대한 테스트)

Etc.

x

@l5x5l l5x5l linked an issue Jan 19, 2025 that may be closed by this pull request
11 tasks
@l5x5l l5x5l requested a review from jiwon2724 January 19, 2025 12:26
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.

[REFACTOR] 코드 리펙토링
1 participant