-
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/#71] 설정창 / 로그아웃, 회원탈퇴 api 연결 #78
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.
너무너무 수고 많아씁니다!!!!!👍👍👍👍👍👍👍👁️👁️
@PATCH("api/users/signout") | ||
suspend fun patchSignOut(): SignOutResponseDto | ||
|
||
@DELETE("api/users/withdraw") | ||
suspend fun deleteWithDraw(): NullableBaseResponse<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.
patch부터 delete까지................미춰따
private fun observeUserSignOutState() { | ||
viewModel.userSignOutState.flowWithLifecycle(lifecycle).onEach { state -> | ||
when (state) { | ||
true -> restartApp(requireContext()) | ||
false -> toast(getString(R.string.server_error)) | ||
null -> {} | ||
} | ||
}.launchIn(lifecycleScope) | ||
} |
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.
설정 뷰 야무지게 로직 구현해주셔서 감사합니당~!!!!!
|
||
@DELETE("api/users/withdraw") | ||
suspend fun deleteWithDraw(): NullableBaseResponse<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.
DELETE
도 있군여,,,
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.
DB 정보를 Delete 할때 쓴답니다~
context.startActivity(Intent.makeRestartActivityTask(componentName)) | ||
Runtime.getRuntime().exit(0) | ||
} |
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.
zz굿
override suspend fun deleteWithDraw(): Result<String?> = kotlin.runCatching { | ||
settingDataSource.deleteWithDraw().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.
kotlin 안예쁘니깐 빼주세요 ㅎㅎ
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.
뺐습ㄴ디ㅏ ㅎㅎㅎㅎ
import retrofit2.http.PATCH | ||
|
||
interface SettingService { | ||
@PATCH("api/users/signout") | ||
suspend fun patchSignOut(): SignOutResponseDto | ||
|
||
@DELETE("api/users/withdraw") | ||
suspend fun deleteWithDraw(): NullableBaseResponse<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.
? 뺴도 돌아갈걸요?
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
📢 To Reviewers