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

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

Open
wants to merge 4 commits into
base: develop
Choose a base branch
from
Open

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