[#511] Bookmark 페이지 추가#523
Conversation
- 2025/app 브랜치와 컴포넌트 네이밍 맞추기 위함
|
|
||
| @Composable | ||
| fun BookmarkScreen( | ||
| fun BookmarkRoute( |
There was a problem hiding this comment.
@taehwandev
2025/app 브랜치 버전과 네이밍을 일치시키기 위함이었습니다~
해당 브랜치 기준으로 요 함수만 BookmarkScreen으로 바꿀까도 생각했었는데요, 이렇게되면 BookmarkScreen이 두개가 되어서, 기존대로 Route 네이밍을 사용하였습니다.
| ) { | ||
| items( | ||
| items = bookmarkItems, | ||
| key = { item -> item.session.id }, |
There was a problem hiding this comment.
contentType 추가해주시면 최적화에 더 좋습니다
There was a problem hiding this comment.
@taehwandev
북마크 리스트의 모든 아이템이 모두 동일한 ViewType(레이아웃 구조)로 보이는데요,
이 경우에도 contentType을 지정하는 것이 성능에 의미가 있는지 궁금합니다.
| _errorFlow.emit(throwable) | ||
| }.launchIn(viewModelScope) | ||
| } | ||
| } |
There was a problem hiding this comment.
@taehwandev
저도 이부분 의문이긴 했었는데요~
해당 Flow Builder를 통해 만들어진 Flow를 사용하지 않는 시점에서, 해당 코드는 오버헤드라 생각합니다ㅎㅎ
기존 코드베이스를 기반으로 빠르게 Feature를 따다 보니 큰 리스크가 없다고 판단하여 적용했던 코드였습니다.
요건 try-catch나 runCatching으로 개선해볼게요~!
북마크 삭제 시 `flow`를 생성하여 예외를 처리하던 방식에서 `runCatching`을 사용하도록 수정했습니다. 이를 통해 코드를 간소화하고 가독성을 향상시켰습니다.
|
@easternkite 안녕하세요. 현생 이슈로 리뷰가 늦어지고 있는데, 내일 중 리뷰 완료 하겠습니다🙏 |
|
@workspace 네넵 |
workspace
left a comment
There was a problem hiding this comment.
@easternkite 수고하셨습니다! ㅎㅎ 기존 코드를 kotlin multiplatform 코드로 옮기는 측면에서 살펴봤는데 너무 잘 작업해주셨습니다. 머지하겠습니다!
f2cdcdb
into
droidknights:2025/compose-multiplatform
@workspace |
Issue
Overview (Required)
2025/app브랜치 코드 기반으로 작성하였습니다.)Fix
Screenshot