-
Notifications
You must be signed in to change notification settings - Fork 2
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
[UI/#28] 유형검사 결과 페이지 #34
Conversation
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.
좋아요~~~~ 하이볼 한잔 더드리겠습니다 ~
@SerialName("status") | ||
val status: Int, | ||
@SerialName("message") | ||
val message: String, | ||
@SerialName("data") | ||
val data: Data, | ||
) { |
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.
BaseResponse를 사용하면 더 효과적일 것 같습니다 ~
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.
BaseResponser가 없어서 제가 만들었습니다~
물론 적용도 했구요 우하하
@@ -1,26 +1,35 @@ | |||
package com.going.presentation.auth | |||
|
|||
import android.content.Context | |||
import android.util.Log |
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.
허헣... 삭제했어요 ^&^
initFinishBtnClickListener() | ||
} | ||
|
||
private fun bindTendencyInfo() { |
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.
허허 적용 했습니다!
|
||
private fun setBulletPoint(text: String): SpannableString { | ||
val string = SpannableString(text) | ||
string.setSpan(BulletSpan(10), 0, text.length - 1, Spanned.SPAN_EXCLUSIVE_EXCLUSIVE) |
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.
맨 뒤에 Spanned의 테그들의 효과들에 대해서도 한번 알아보면 좋을 것 같아요 ~
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.
공부 완!
@@ -0,0 +1,11 @@ | |||
<?xml version="1.0" encoding="utf-8"?> | |||
<shape xmlns:android="http://schemas.android.com/apk/res/android" |
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.
변경 완~~
깜빡했네욤 우훟
@@ -123,7 +123,7 @@ | |||
app:layout_constraintEnd_toEndOf="@id/et_onboarding_profile_setting_info" | |||
app:layout_constraintTop_toBottomOf="@id/et_onboarding_profile_setting_info" /> | |||
|
|||
<androidx.appcompat.widget.AppCompatButton | |||
<androidx.appcompat.widget.AppCompatEditText |
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.
EditText로 바뀐 이유가 있나요~~~?
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.
헉 실수입니다.. 수정할게요!!
loginRepositoryImpl |
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.
코틀린스러운 코드네요 ~~ LGTM
⛳️ Work Description
📸 Screenshot
📢 To Reviewers