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

마이페이지 리팩토링 #269

Merged
merged 46 commits into from
Jul 21, 2023
Merged

Conversation

miinjung
Copy link
Contributor

@miinjung miinjung commented Jun 28, 2023

현재 develop 을 머지하면 gradle 에 이상이 생겨서 우선적으로 commit 통한 diff 남기기 용으로 PR 올림

미 진행 상황

  • dispose 방식
    -> RxLifecycle / autoDispose 둘중에 하나 선택해서 커스텀이 필요함. 현재의 방식은 아직 검증이 안됨.

@miinjung miinjung added the duplicate This issue or pull request already exists label Jun 28, 2023
@miinjung miinjung requested a review from a team as a code owner June 28, 2023 13:09
@miinjung miinjung self-assigned this Jun 28, 2023
@miinjung miinjung changed the base branch from develop to feature/minjung/230422-mypage June 28, 2023 13:16
@miinjung miinjung changed the base branch from feature/minjung/230422-mypage to develop June 28, 2023 13:16
choiminjung added 2 commits July 3, 2023 21:05
# Conflicts:
#	app/src/main/java/org/sopt/official/feature/mypage/MyPageActivity.kt
#	app/src/main/java/org/sopt/official/feature/mypage/MyPageViewModel.kt
#	app/src/main/java/org/sopt/official/feature/mypage/signOut/SignOutActivity.kt
#	app/src/main/java/org/sopt/official/feature/mypage/signOut/SignOutViewModel.kt
#	app/src/main/java/org/sopt/official/feature/mypage/soptamp/nickName/ChangeNickNameActivity.kt
#	app/src/main/java/org/sopt/official/feature/mypage/soptamp/nickName/ChangeNickNameViewModel.kt
#	app/src/main/java/org/sopt/official/feature/mypage/soptamp/sentence/AdjustSentenceActivity.kt
#	app/src/main/java/org/sopt/official/feature/mypage/soptamp/sentence/AdjustSentenceViewModel.kt
#	app/src/main/res/values/strings.xml
Copy link
Member

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

Dispose 방식은 이정도면 될 것 같음. 만약에 추가적으로 autoDispose가 필요하면 AutoDisposableFragment/Activity같은걸 만들면 될 것 같아

@l2hyunwoo l2hyunwoo merged commit 95f2fe1 into develop Jul 21, 2023
@l2hyunwoo l2hyunwoo deleted the feature/minjung/mypage-refactor branch July 21, 2023 03:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
duplicate This issue or pull request already exists size/XL
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants