Closed workspace closed 5 months ago
@laco-dev 안녕하세요. 현재 reference-media3
에 widget 업데이트가 포함 안되어있는데요, 업데이트 해주시면 다시 싱크 맞추겠습니다. 이제 합치기 위한 준비가 되어 리뷰 부탁드립니다. 변경 사항이 큰 것 같아 overview와 몇몇 comment를 달아뒀습니다.
내용이 많아서 아직 모든 부분을 확인하지는 못했습니다. 😭 행사 Session과 미디어 Session이 이름이 중복되서 헷갈리는 상황이 많이 왔는데 이 이름이 일반적으로 미디어쪽의 세션이라고생각하는 상황이 많을까요?
안녕하세요. 상세한 리뷰 감사드립니다.
발표를 위해 구현했던 사항을 한번에 보내 pr이 너무 커지게 된 점 죄송합니다. pr overview에 남겨둔 방향으로 접근하시면 좀 더 용이한 리뷰가...🥲🥲🥲
미디어 Session으로 쓰이는 이름들은 보통 Player를 초기화 하는 쪽에서만 쓰이고 나머지는 행사 Session으로 봐주시면 됩니다.
저도 오랜만에 보니깐 충분히 헷갈릴 만한 상황 같네요. Session이라는 이름이 겹치는 이번 프로젝트에 한해서는 import alias 등으로 명확히 MediaSession* 이란 접두사를 붙여볼 수도 있을 것 같네요.
리뷰 달아주신 사항들은 개별로 커멘트 남기겠습니다.
안녕하세요. @laco-dev 여러가지가 겹치며 업데이트가 늦어진 점 사과드립니다. 🥲 마지막으로 리뷰 하신 후 주요 업데이트 사항 요약은 아래와 같습니다.
더불어, 현재 json file을 이용하여 api 구현을 해두었기 때문에 pr을 그대로 빌드하면 Video 정보가 조회되지 않아 재생을 할 수 없습니다. 혹시 동작을 확인하시고자 한다면 GithubRawApi
의 getSessions
request 경로에서 /owner/repository/branch
를 제 folk 것으로 임시로 변경해야 확인할 수 있습니다.
// GithubRawApi.kt
// 변경 전
@GET("/droidknights/DroidKnights2023_App/reference-media3/core/data/src/main/assets/sessions.json")
// 변경 후
@GET("/workspace/DroidKnights2023-app-with-media3/media3/core/data/src/main/assets/sessions.json"
또한, 몇가지 업데이트가 추가될 수 있는데, 이건 그냥 제 folk에서 계속 업데이트 하게될 것 같습니다. 혹시 #265 를 작업하신다면 제 folk repository(https://github.com/workspace/DroidKnights2023-app-with-media3) 도 병기해주시면 감사하겠습니다!
머지합니다.
Issue
Overview
라이브러리 업그레이드
tv, wear-os, automotive app overview
tv, wear-os, automotive feature overview
core:playback overview
발표자료
+) 원래 media3 구현에 맞춰 개인 repo에 가지고 있으려던 것인지라 에잇! 하며 허술한 부분이 있을 수 있습니다. 개선 방향 제시해주시면 감사하게 반영해보겠습니다 ㅎㅎ