Issue

• close #2

Overview (Required)

• ~기능 개발을 하다가 궁금한 점들이 있어, 코드리뷰를 받고자 먼저 PR을 날리게 되었습니다.~
• ~Question~
  • ~1. BindingAdapter 내에서 suspend 함수를 사용하고자 CoroutineScope이 필요했는데 Dispatchers.Main.immediate을 사용하기에는 Fragment의 View가 destroy 되어도 함수가 계속 작동할 수 있을 것 같아 정상적인 작동이 보장되지 않을 것 같았습니다. 그래서 findViewTreeLifecycleOwner()?.lifecycleScope을 사용하려 했으나 findViewTreeLifecycleOwner가 null이 나오더군요. Fragment의 뷰가 다 만들졌는데 RecyclerView의 ViewTreeLifecycleOwner가 Null이 뜨는 이유가 궁금합니다. 아래 코드를 참고를 했는데도 불구하고 원인이 이해가 안되었습니다. 제가 이해하기로는 View의 parent를 탐색하여 Null이 아닌 LifecycleOwner를 찾는 것이었습니다.~
  • ~2. 위의 문제로 인해 Fragment의 LifecycleScope을 바인딩 어댑터 부분에 주입을 하였는데, Fragment의 onDestroyView가 호출된 이후에는 LifecycleScope가 null이 될텐데 이 때문에 NullPointerExcpetion이 발생될 수 있을 것 같은데 제 예상이 맞은가요?~
  • ~3. 이슈에 주신 링크를 보았는데 ComposeView를 기존 패턴에서 사용하기 위해서는 onCreateViewHolder와 onBindViewHolder를 모두 사용해야할 것 같은데, ListBindingAdapter 대신 ComposeAdapter, ComposeViewHolder와 같은 Util Adapter를 제작해서 사용하는 건 어떤지 의견을 듣고 싶습니다.~

    @Nullable
    public static LifecycleOwner get(@NonNull View view) {
        LifecycleOwner found = (LifecycleOwner) view.getTag(R.id.view_tree_lifecycle_owner);
        if (found != null) return found;
        ViewParent parent = view.getParent();
        while (found == null && parent instanceof View) {
            final View parentView = (View) parent;
            found = (LifecycleOwner) parentView.getTag(R.id.view_tree_lifecycle_owner);
            parent = parentView.getParent();
        }
        return found;
    }

~Updated 21/08/28~

• ~아래의 이슈만 해결이 된다면 머지를 할 수 있을 것 같습니다.~ ~Q) NetworkImage 클래스에서 rememberImagePainter를 사용하는데 이미지가 로드가 되지 않고 painter.state가 항상 empty가 되는 현상을 발견했습니다. 로컬에서 이미지를 받아올 때에는 이미지 로드가 잘 되는 것을 확인을 했었습니다.~
• ~로깅을 해봤을 때 painter.request의 dispatcher가 Dispatchers.IO로 되어있어서~

internal fun NetworkImage(
    imageUrl: String,
    nonSuccessTintColor: Color,
    modifier: Modifier = Modifier,
    builder: ImageRequest.Builder.() -> Unit = {
        placeholder(R.drawable.ic_android_white)
        error(R.drawable.ic_android_white)
        dispatcher(Dispatchers.Main.Immediate)
    },
)

~로도 변경을 해보았으나 이 역시 실행이 안되었습니다.~ ~디버그를 해도 감이 잡히지 않아서 질문드립니다 🙏🏻 @Pluu~

Links

• ComposeAdapter 참고자료: https://github.com/android/sunflower/blob/compose_recyclerview/app/src/main/java/com/google/samples/apps/sunflower/adapters/PlantAdapter.kt

Screenshot 2021.09.26 UPDATED

Before | After :--: | :--: |

Kind (Required)

• Improvement

Overview (Required)

• 세션 노출은 시간-세션명 순서대로 노출되어야합니다

Current

• 현재 세션 리스트는 json에 저장된 순서대로 노출됩니다.

| 참고 자료 | 참고 자료 | | :---: | :---: | | | |

Links

• https://github.com/droidknights/DroidKnights2021_App/blob/main/domain/src/main/java/com/droidknights/app2021/domain/usecase/session/GetSessionsUseCase.kt#L28
• https://github.com/droidknights/DroidKnights2021_App/blob/main/app/src/main/assets/sessions.json#L12

Issue

• close #34

Overview (Required)

• 동일한 컴파일러 설정이 빌드스크립트에 중복되어 있습니다. 한곳으로 모아두는 것이 추후 관리하는데 좋을 것 같습니다.
• 현재 kotlin DSL은 아직 개발중이며, API들이 제대로 노출되어 있지 않습니다. 그래서 Groovy DSL을 이용해서 통합했습니다.

Links

• https://developer.android.com/studio/releases/gradle-plugin-roadmap

Issue

• close #3

Overview (Required)

• TextView에 Date만 뜰 수 있도록 BindingAdapter 변경
• 이때, isEndEvent는 사용되지 않아 parameter 삭제

Screenshot

Before | After :--: | :--: |

Kind (Required)

• Improvement

Overview (Required)

• 스폰서 영역이 자동 스크롤이 지원되어야 합니다.
• 자동 스크롤 방향은 오른쪽에서 왼쪽으로 흐르기가 되어야합니다
• (Option) 해당 영역을 Compose로 개선
  • (Required) 외부에서 사용하는 RecyclerView.Adapter 및 ViewHolder 패턴을 유지해야합니다

Link

• https://developer.android.com/jetpack/compose/interop/compose-in-existing-ui
• https://proandroiddev.com/infinite-auto-scrolling-lists-with-recyclerview-lazylists-in-compose-1c3b44448c8

Issue

• close #4

Overview (Required)

• Chrome Custom Browser 라이브러리 추가
• Context 확장함수 기반으로 string을 받아 웹뷰를 띄우도록 코드 구현
• 기존 아답터에서 호출되던 코드를 마이그레이션

Links

• https://mvnrepository.com/artifact/androidx.browser/browser/1.3.0

Screenshot

Before | After :--: | :--:

https://user-images.githubusercontent.com/27615921/130397875-fa16fccd-6302-4606-8095-45a804e07c97.mp4

https://user-images.githubusercontent.com/27615921/130397883-b40e830e-57f9-40a1-b965-f875c1739dd1.mp4

Issue

• close #24

Overview (Required)

• gradle kotlin dsl로 마이그레이션 했습니다.
• arctic fox 2020.3.1 에서 정상동작하는 것을 확인했습니다. 최초 적용때는 일부 에러표시가 있었는데 캐시 삭제 를 한번만 진행하면 이후로는 괜찮았습니다.
• 변경점이 많아서 file change 인식이 안되어서 커밋을 나누었습니다. 변경점은 fed3ea9 로 몰아넣었습니다.
• jcenter에 관한 내용이 남아있어서 지웠습니다.
• 라이브러리 모듈의 버전 정보를 지웠습니다.

Links

• kotlin 1.5부터는 jvm 1.8이 기본옵션이라 gradle 설정에서 지웠습니다.

Kind (Required)

• Other

Overview (Required)

• 정의되었지만, 사용하지않는 Qualifier는 삭제해야하는 대상입니다.

Links

• https://github.com/droidknights/DroidKnights2021_App/blob/main/shared/src/main/java/com/droidknights/app2021/shared/di/CoroutinesQualifiers.kt#L23

Issue

• close #1

Overview (Required)

• ImageView를 Material ShapeableImageView로 변환
• app:shapeAppearanceOverlay attribute에 corner 속성을 적용

Screenshot

Before | After :--: | :--: |

Kind (Required)

• Improvement

Overview (Required)

• json에 있는 트랙정보는 String으로 정의되어 있어 DTO 변환시 값에 영향을 받습니다. 유연한 대응을 위해서 직렬화 개선이 필요합니다.
• (Required) Track1, track1은 Room.Track1으로 매칭됩니다.
• (Required) 트랙정보가 올바르게 매칭되지 않을시 Room.Etc로 매칭됩니다.

Links

• KSerializer 사용
  • https://github.com/Kotlin/kotlinx.serialization/blob/master/guide/example/example-builtin-03.kt#L11
  • https://github.com/Kotlin/kotlinx.serialization/blob/master/core/commonMain/src/kotlinx/serialization/builtins/LongAsStringSerializer.kt
• DTO 정보 : https://github.com/droidknights/DroidKnights2021_App/blob/main/data/src/main/java/com/droidknights/app2021/data/model/SessionData.kt#L22
• Room 정보 : https://github.com/droidknights/DroidKnights2021_App/blob/main/shared/src/main/java/com/droidknights/app2021/shared/Domain.kt#L20

Kind (Required)

• Improvement

Overview (Required)

• 트랙 순서는 Track1 > Track2 > Etc 순서로 노출되어야합니다.
• 현재는 트랙이 노출되는 순서는 json에 track이 저장된 순서에 영향을 받습니다.

Info

• 세션의 트랙 정보 : https://github.com/droidknights/DroidKnights2021_App/blob/main/app/src/main/assets/sessions.json#L80
• (Required) json의 순서를 수정해서는 안됩니다.

Current

Links

• https://github.com/droidknights/DroidKnights2021_App/blob/main/features/schedule/src/main/java/com/droidknights/app2021/schedule/ScheduleScreen.kt#L32

Kind (Required)

• Other

Overview (Required)

• 일부 프로필 이미지는 원형으로 노출되지 않고, 좌우가 잘린 형태로 노출되고 있습니다.
• 프로필로 노출되는 이미지는 원형 형태로 노출되어야합니다.

| Screenshot | Screenshot | | :--: | :--: | | | |