Skip to content

[Feature] #86 미분류 링크 분류 화면 구현#87

Merged
l5x5l merged 7 commits intodevelopfrom
feature/#86-uncategorized_link_screen
Dec 15, 2024
Merged

[Feature] #86 미분류 링크 분류 화면 구현#87
l5x5l merged 7 commits intodevelopfrom
feature/#86-uncategorized_link_screen

Conversation

@l5x5l
Copy link
Contributor

@l5x5l l5x5l commented Dec 14, 2024

Key Changes

미분류 포킷 분류 화면을 구현하고
LinkCard의 세부 UI를 수정했습니다. (배경색 및 안읽음 표시 수정)
Resolves: #86

PR 유형

어떤 변경 사항이 있나요?

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

To Reviewers

  • 미분류 화면의 경우 별도의 feature:uncategorized 모듈에 생성했습니다.
  • 코드 보면서 이해가 어렵거나 가시성이 안좋은 부분 있다면 피드백 부탁드립니다!

PR Checklist

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

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

Etc.

기존 화면을 구현할 때는Preview를 사용하기 위해서 [화면이름]ScreenContainer에 ViewModel을 받고
[화면이름]Screen에 여러 메서드들을 인자로 전달하는 방식을 사용했는데, 이렇게 하다보니 너무 인자 개수가 많아지는 것 같아서
이번에는 ViewModel을 인터페이스로 선언하여 사용해봤습니다 (UncategorizedViewModel 참고)
이 방식 괜찮은 것 같으면 앞으로 [화면이름]ScreenContainer로 구현한 부분은 모두 ViewModel 인터페이스를 사용하는 방식으로 리펙토링 진행하려고 합니다!

@l5x5l l5x5l linked an issue Dec 14, 2024 that may be closed by this pull request
[FEATURE] 미분류 포킷 분류 화면 구현 #86
Closed
2 tasks
@l5x5l l5x5l requested a review from jiwon2724 December 14, 2024 09:06
jiwon2724
jiwon2724 reviewed Dec 15, 2024
View reviewed changes
Copy link
Member

@jiwon2724 jiwon2724 left a comment

Choose a reason for hiding this comment

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

고생하셧슴미다~~bb 하나 빼고 따로 남길 부분은 없어욤

feature/uncategorized/src/main/java/pokitmons/pokit/uncategorized/UncategorizedViewModel.kt
import pokitmons.pokit.uncategorized.model.UncategorizedScreenState
import javax.inject.Inject

interface UncategorizedViewModel {
Copy link
Member

@jiwon2724 jiwon2724 Dec 15, 2024

Choose a reason for hiding this comment

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

interface 네이밍에 ViewModel 워딩은 의도된건가욤?

Copy link
Member

@jiwon2724 jiwon2724 Dec 15, 2024 •
edited
Loading

Choose a reason for hiding this comment

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

아 etc를 방금봄ㅋㅋㅋㅋ

@l5x5l l5x5l requested a review from jiwon2724 December 15, 2024 09:44
jiwon2724
jiwon2724 approved these changes Dec 15, 2024
View reviewed changes
Copy link
Member

@jiwon2724 jiwon2724 left a comment

Choose a reason for hiding this comment

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

해당 방법으로 한 번 추라이 해보졍

@l5x5l l5x5l merged commit 5e0ef50 into develop Dec 15, 2024
1 check passed
jiwon2724 pushed a commit that referenced this pull request Jan 11, 2025
@l5x5l @jiwon2724
[Feature] #86 미분류 링크 분류 화면 구현 (#87)
ba078a9 
* [BASE] #86 feature:uncategorized 모듈 생성

* [FEATURE] #86 링크 분류하기 화면 UI 구현

* [FEATURE] #86 홈 화면과 링크 분류하기 화면간 이동 로직 구현

* [FEATURE] #86 링크 분류하기 화면에서 사용하는 API 연결

* [FEATURE] #86 링크 분류하기 화면에서 링크 제거 및 포킷 변경 발생 이후 다시 홈 화면으로 돌아왔을 때 refresh하는 기능 구현

* [UI] #86 LinkCard 세부 디자인 수정

* [CHORE] #86 ktlint 적용
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@jiwon2724 jiwon2724 jiwon2724 approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

[FEATURE] 미분류 포킷 분류 화면 구현

2 participants

@l5x5l @jiwon2724