Skip to content

[Feature/#92] 프로필 수정 구현#95

Open
kkk5474096 wants to merge 4 commits intodevelopfrom
feature/#92
Open

[Feature/#92] 프로필 수정 구현#95
kkk5474096 wants to merge 4 commits intodevelopfrom
feature/#92

Conversation

@kkk5474096
Copy link
Contributor

Related Issues

What Did You Do?

  • 프로필 수정 구현

@kkk5474096 kkk5474096 added feature 💡 새로운 Feature 나 작업 사항 강민 🦖 나는 강민! labels Jan 5, 2023
@kkk5474096 kkk5474096 requested a review from a team January 5, 2023 09:52
@kkk5474096 kkk5474096 self-assigned this Jan 5, 2023
Copy link
Contributor

@l2hyunwoo l2hyunwoo left a comment

Choose a reason for hiding this comment

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

고생하셨습니다.

import androidx.core.widget.doAfterTextChanged
import androidx.lifecycle.flowWithLifecycle
import androidx.lifecycle.lifecycleScope
import com.bumptech.glide.Glide
Copy link
Contributor

Choose a reason for hiding this comment

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

Glide 뗄까...

Comment on lines +53 to +59
binding.ivNicknameClear.setOnClickListener {
viewModel.nickname.value = ""
}

binding.ivCameraClear.setOnClickListener {
viewModel.cameraName.value = ""
}
Copy link
Contributor

Choose a reason for hiding this comment

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

이거 함수로 빼보자

Comment on lines +62 to +63
val intent = Intent(Intent.ACTION_GET_CONTENT)
intent.type = "image/*"
Copy link
Contributor

Choose a reason for hiding this comment

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

이거 사용합시다

Comment on lines +78 to +83
setBackgroundColor(
if (it) colorOf(R.color.fillin_red) else Color.parseColor("#474645")
)
setTextColor(
if (it) colorOf(R.color.fillin_black) else Color.parseColor("#6F6F6F")
)
Copy link
Contributor

Choose a reason for hiding this comment

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

Color.parseColor는 최대한 지양해보자!

Comment on lines +115 to +121
imageUri?.let {
if (Build.VERSION.SDK_INT >= Build.VERSION_CODES.P) {
viewModel.setImageBitmap(ImageDecoder.decodeBitmap(ImageDecoder.createSource(contentResolver, imageUri)))
} else {
viewModel.setImageBitmap(MediaStore.Images.Media.getBitmap(contentResolver, imageUri))
}
}
Copy link
Contributor

Choose a reason for hiding this comment

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

ContentUriRequestBody 사용하는 쪽으로 가보는건 어떨깝쇼

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

feature 💡 새로운 Feature 나 작업 사항 강민 🦖 나는 강민!

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[FEAT] 프로필 수정 구현

2 participants