-
Notifications
You must be signed in to change notification settings - Fork 4
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
Update Calls SDK v1.12.1 #4
Conversation
distributionBase=GRADLE_USER_HOME | ||
distributionPath=wrapper/dists | ||
distributionUrl=https\://services.gradle.org/distributions/gradle-7.5-bin.zip |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
가장 최신으로 올리면 문제되나요?!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
최신으로 올리면 아마 다른 라이브러리들도 같이 올려야할텐데요, 최대한 calls sdk랑 버전이 차이나지 않는게 좋을 것 같아 8.5 정도로 올렸습니다. 8.5는 현재 버전의 Android Studio가 빌드할 때 필요한 최소 버전입니다.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
요거 android studio에서 가이드하는 버전으로 올렸습니다. 8.10.2로 더 올릴 수는 있는데 stable하게 쓰려면 studio에서 제안하는 버전으로 올리는게 좋을 것 같았어요.(8.9도 거의 최신 버전이라 이 정도도 괜찮지 않을까 싶어요!) 16793f4
gradle version 문서
app/build.gradle
Outdated
@@ -6,13 +6,13 @@ plugins { | |||
} | |||
|
|||
android { | |||
compileSdkVersion 30 | |||
compileSdkVersion 33 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
요것도 34로 최신 되나요~? 35도 될라나요?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
calls sdk랑 버전을 맞춰야할 것 같아 33으로 맞췄습니다! 34나 35로 올리면 여기서 실행되는 SDK의 targetSDK도 같이 올라가서 되도록이면 calls sdk 버전과 맞추는게 어떨까 싶습니다!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
요게 샘플이라 고객들 기준으로 보는게 좋을 것 같아서요. 고객들은 최신 버전에서 사용하는 케이스가 많을테니 문제 없으면 올리는게 좋을 것 같아요.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
수정했습니다! 1223129
build.gradle
Outdated
@@ -1,14 +1,14 @@ | |||
// Top-level build file where you can add configuration options common to all sub-projects/modules. | |||
buildscript { | |||
ext.kotlin_version = "1.4.31" | |||
ext.kotlin_version = "1.6.10" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
코틀린도 더 올릴 수 있으면 올리시죠!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Kotlin version도 1.9.21로 올렸는데 최신은 아니고 kapt plugin이라는 현 샘플앱에서 사용하는 plugin이 동작하는 최소 버전으로 올렸습니다 28474d1
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
최신으로 수정했습니다! 24a8023
build.gradle
Outdated
ext.nav_version = "2.3.4" | ||
ext.camerax_version = "1.0.0-rc03" | ||
repositories { | ||
google() | ||
mavenCentral() | ||
} | ||
dependencies { | ||
classpath "com.android.tools.build:gradle:4.1.2" | ||
classpath "com.android.tools.build:gradle:7.4.2" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
요것도 되면!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
옙 요거 8.1.2로 올렸는데 최신은 아니고 calls sdk랑 버전 같도록 올렸습니다!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
gradle version과 같이 올렸습니다! 16793f4
… not affected foreground service and pending intent restrictions.
Background
Calls SDK v1.12.1이 groupcall 샘플에서 빌드가 안된다는 이슈가 있어 sdk version을 올림과 동시에 필요한 것들을 올렸습니다.
이전에 엘리엇이 Gradle Version을 올리려던 PR의 내용들을 참고해서 Kotlin version과 Gradle version을 Calls SDK 버전과 맞게 올렸습니다.