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

Feat/cached network image #219

Merged
merged 13 commits into from
May 21, 2024
Merged

Feat/cached network image #219

merged 13 commits into from
May 21, 2024

Conversation

somarok
Copy link
Contributor

@somarok somarok commented May 20, 2024

:: 최근 작업 주제 (하나 이상의 주제를 선택해주세요.)

  • 기능 추가
  • 리뷰 반영
  • 리팩토링
  • 버그 수정
  • 컨벤션 수정

:: 구현 목표 (해당 브랜치에서 구현하고자 하는 하나의 목표를 설정합니다.)

  • 이미지 로딩 개선
  • 컨벤션 및 클린아키텍쳐 준수

:: 구현 사항 설명 (작업한 내용을 상세하게 기록합니다.)

  1. Image.network 위젯을 커스텀한 CachedImageWidget으로 변경
  2. GetIt 주입 로직 잘못 된 부분 수정
  3. dataBox HiveWrapper 필드로 이동
  4. 파일, 폴더명 컨벤션에 맞춰 변경
  5. 불필요한 코드 삭제

:: 성장 포인트 (해당 기능을 구현하며 고민했던 사항이나 새로 알게된 부분, 어려웠던 점 등을 작성합니다.)


:: 기타 질문 및 특이 사항

@somarok somarok added 🛠️ Refactor 리팩토링 [PR] 리뷰요청 리뷰 요청 드립니다. labels May 20, 2024
@somarok somarok self-assigned this May 20, 2024
Copy link
Collaborator

@SeongJongHo SeongJongHo left a comment

Choose a reason for hiding this comment

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

LGTM👍

Copy link
Collaborator

@hoogom88 hoogom88 left a comment

Choose a reason for hiding this comment

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

LGTM👍

Copy link
Collaborator

@gyubit gyubit left a comment

Choose a reason for hiding this comment

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

LGTM 👍🏻👍

@hoogom88 hoogom88 merged commit 19fb77a into dev May 21, 2024
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
[PR] 리뷰요청 리뷰 요청 드립니다. 🛠️ Refactor 리팩토링
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants