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

#02 인기영화 목록 보기 및 영화 상세보기 화면 제공 #33

Open
wants to merge 86 commits into
base: 2giwon
Choose a base branch
from

Conversation

2giwon
Copy link
Collaborator

@2giwon 2giwon commented Apr 7, 2020

리뷰 부탁드립니다

#1 프로젝트 초기 구성
@2giwon 2giwon self-assigned this Apr 7, 2020
@2giwon 2giwon requested review from Hoyuo and taeiim April 7, 2020 15:25

android {
compileSdkVersion 29
buildToolsVersion "29.0.3"
Copy link
Collaborator

Choose a reason for hiding this comment

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

buildToolsVersion은 없어도 됩니다.
한번 찾아보시면 공부하는데 도움이 될꺼같습니다.

Copy link
Collaborator Author

@2giwon 2giwon Apr 8, 2020

Choose a reason for hiding this comment

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

1f24273

gradle 3.x 이후 부터 적용하지 않아도 된다고 하는데 이게 맞나요?

Copy link
Collaborator

Choose a reason for hiding this comment

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

네 맞습니다

import androidx.recyclerview.widget.ListAdapter
import androidx.recyclerview.widget.RecyclerView

abstract class BaseRecyclerView {
Copy link
Collaborator

Choose a reason for hiding this comment

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

BaseRecyclerView class 가 필요한지가 의문입니다

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

맞네요 습관처럼 만들다보니 인지하지 못하고 있었습니다. 수정했습니다

f624c00

@Hoyuo
Copy link
Collaborator

Hoyuo commented Apr 8, 2020

disposable 관리에 대해서는 좀 더 이야기가 필요 하다고 생각됩니다

@2giwon 2giwon requested a review from Hoyuo April 8, 2020 05:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants