Skip to content

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
@l5x5l l5x5l merged commit 4e822e3 into develop Feb 9, 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.

[REFACTOR] 코드 리펙토링

2 participants