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

[UI/#28] 유형검사 결과 페이지 #34

Merged
merged 23 commits into from
Jan 8, 2024

Conversation

chattymin
Copy link
Member

@chattymin chattymin commented Jan 7, 2024

⛳️ Work Description

  • UI 구현
  • 블러 구현
  • 버튼 구현
  • 세로 점선 구현
  • 로그인 API 연결
  • 그...외....다수....

📸 Screenshot

image

📢 To Reviewers

  • 세로 점선이 복병이었네요... 너비를 넓게해야 나타나길래 그냥 100 했습니다 ㅋㅋ
  • 리펙토링 필수인 코드 같습니다...
  • 갑작스러운 요청땜에 로그인 API를 연결했습니다 푸하하

Copy link
Member

@Marchbreeze Marchbreeze left a comment

Choose a reason for hiding this comment

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

좋아요~~~~ 하이볼 한잔 더드리겠습니다 ~

Comment on lines 9 to 15
@SerialName("status")
val status: Int,
@SerialName("message")
val message: String,
@SerialName("data")
val data: Data,
) {
Copy link
Member

Choose a reason for hiding this comment

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

BaseResponse를 사용하면 더 효과적일 것 같습니다 ~

Copy link
Member Author

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
Copy link
Member

Choose a reason for hiding this comment

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

로그 조심해주세요 ~

Copy link
Member Author

Choose a reason for hiding this comment

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

허헣... 삭제했어요 ^&^

initFinishBtnClickListener()
}

private fun bindTendencyInfo() {
Copy link
Member

Choose a reason for hiding this comment

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

스코프 함수를 활용해주세요 ~

Copy link
Member Author

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)
Copy link
Member

Choose a reason for hiding this comment

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

맨 뒤에 Spanned의 테그들의 효과들에 대해서도 한번 알아보면 좋을 것 같아요 ~

Copy link
Member Author

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"
Copy link
Member

Choose a reason for hiding this comment

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

네이밍 확인해주세요 ~

Copy link
Member Author

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
Copy link
Member

Choose a reason for hiding this comment

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

EditText로 바뀐 이유가 있나요~~~?

Copy link
Member Author

Choose a reason for hiding this comment

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

헉 실수입니다.. 수정할게요!!

Comment on lines +24 to +25
loginRepositoryImpl
Copy link
Member

Choose a reason for hiding this comment

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

코틀린스러운 코드네요 ~~ LGTM

@chattymin chattymin merged commit 2783bec into develop Jan 8, 2024
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[UI] 유형검사결과 UI 구현
2 participants