-
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
[FEAT/#114] 프로필 조회 뷰 / 서버통신 구현 #125
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.
수고해써요!!!!! 마지막까지 파이티잉
app/src/main/AndroidManifest.xml
Outdated
android:name="com.going.presentation.setting.SettingActivity" | ||
android:exported="false" | ||
android:exported="true" |
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.
false로 해주세욤
@@ -95,7 +95,11 @@ class EnterTripActivity : BaseActivity<ActivityEnterTripBinding>(R.layout.activi | |||
} | |||
|
|||
is UiState.Failure -> { | |||
toast(getString(R.string.server_error)) | |||
if (state.msg == "존재하지 않는 여행입니다.") { |
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.
요거 string 추출 가능한가용
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.
넵 추출해서 수정했습니답
binding.run { | ||
tvProfileName.text = name | ||
tvProfileOneLine.text = intro | ||
|
||
val profileImage = when (number) { | ||
0 -> R.drawable.img_profile_6 | ||
1 -> R.drawable.img_profile_1 | ||
2 -> R.drawable.img_profile_2 | ||
3 -> R.drawable.img_profile_4 | ||
4 -> R.drawable.img_profile_8 | ||
5 -> R.drawable.img_profile_5 | ||
6 -> R.drawable.img_profile_7 | ||
7 -> R.drawable.img_profile_3 | ||
else -> R.drawable.img_profile_1 | ||
} |
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.
굿굿 자잘한 부분들에 대한 수정만 해주세용 ~
xmlns:app="https://schemas.android.com/apk/res-auto" | ||
xmlns:tools="https://schemas.android.com/tools"> |
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.
하하 수정완이욥
|
||
|
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.
줄바꿈이 2번이네요 ㅎㅎ
if (errorCode == "e4043") { | ||
_tripState.value = UiState.Failure(errorMessage) |
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.
에러코드같은 친구들도 아래에 const val로 따로 빼주는게 좋아요 ~~
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.
말할 부분이 없네요! 너무 잘해버렸자나~
⛳️ Work Description
📸 Screenshot
Screen_Recording_20240115_195811_Gallery.mp4
Screen_Recording_20240115_174759_doorip.mp4
📢 To Reviewers