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

[Feat/#71] 설정창 / 로그아웃, 회원탈퇴 api 연결 #78

Merged
merged 9 commits into from
Jan 11, 2024

Conversation

chattymin
Copy link
Member

⛳️ Work Description

  • 로그아웃 카카오 구현
  • 로그아웃 JWT 구현
  • 회원탈퇴 카카오 구현
  • 회원탈퇴 JWT 구현

📢 To Reviewers

  • 구현 완~

Copy link
Contributor

@crownjoe crownjoe 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 13
@PATCH("api/users/signout")
suspend fun patchSignOut(): SignOutResponseDto

@DELETE("api/users/withdraw")
suspend fun deleteWithDraw(): NullableBaseResponse<String?>
Copy link
Contributor

Choose a reason for hiding this comment

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

patch부터 delete까지................미춰따

Comment on lines 50 to 58
private fun observeUserSignOutState() {
viewModel.userSignOutState.flowWithLifecycle(lifecycle).onEach { state ->
when (state) {
true -> restartApp(requireContext())
false -> toast(getString(R.string.server_error))
null -> {}
}
}.launchIn(lifecycleScope)
}
Copy link
Contributor

Choose a reason for hiding this comment

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

조아욥

Copy link
Member

@leeeyubin leeeyubin 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 11 to 13

@DELETE("api/users/withdraw")
suspend fun deleteWithDraw(): NullableBaseResponse<String?>
Copy link
Member

Choose a reason for hiding this comment

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

DELETE도 있군여,,,

Copy link
Member Author

Choose a reason for hiding this comment

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

DB 정보를 Delete 할때 쓴답니다~

Comment on lines +70 to +72
context.startActivity(Intent.makeRestartActivityTask(componentName))
Runtime.getRuntime().exit(0)
}
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.

앱을 재시작 시키기 위한 코드입니다!

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.

zz굿

Comment on lines 15 to 17
override suspend fun deleteWithDraw(): Result<String?> = kotlin.runCatching {
settingDataSource.deleteWithDraw().data
}
Copy link
Member

Choose a reason for hiding this comment

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

kotlin 안예쁘니깐 빼주세요 ㅎㅎ

Copy link
Member Author

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?>
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.

그래서 수정했습니다!!
상호님도 수정해줘요 ㅎㅎㅎㅎ

@chattymin chattymin merged commit 7d67c4a into develop Jan 11, 2024
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
FEAT ✨ 새로운 기능 구현 동민 🐥
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[FEAT] 설정뷰 / 로그아웃, 회원탈퇴 api 연결
4 participants