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/#38] : 다이얼로그 추가 #40

Merged
merged 9 commits into from
Jan 24, 2025

Conversation

gaeulzzang
Copy link
Contributor

@gaeulzzang gaeulzzang commented Jan 23, 2025

✅ 𝗖𝗵𝗲𝗰𝗸-𝗟𝗶𝘀𝘁

  • merge할 브랜치의 위치를 확인해 주세요.(main❌/develop⭕)
  • 리뷰가 필요한 경우 리뷰어를 지정해 주세요.
  • 리뷰는 PR이 올라오면 최대한 빠르게 진행합니다.
  • P1 단계의 리뷰는 빠르게 확인 후 반영합니다.
  • Approve된 PR은 assigner가 머지하고, 수정 요청이 온 경우 수정 후 다시 push를 합니다.

📌 𝗜𝘀𝘀𝘂𝗲𝘀

📎 𝗪𝗼𝗿𝗸 𝗗𝗲𝘀𝗰𝗿𝗶𝗽𝘁𝗶𝗼𝗻

  • 약속 확정 다이얼로그 추가
  • 로그인(카카오/구글) 다이얼로그 추가
  • 로그인 다이얼로그 SideEffect로 빼놓기
  • 마이페이지 PaddingValues 전달

📷 𝗦𝗰𝗿𝗲𝗲𝗻𝘀𝗵𝗼𝘁

Screen_recording_20250123_145857.mp4

💬 𝗧𝗼 𝗥𝗲𝘃𝗶𝗲𝘄𝗲𝗿𝘀

LoginFailureDialog.kt 삭제했습니당 @youjin09222
LoginViewModel에서 카카오, 구글별로 다이얼로그 상태 따로 관리하는 것보다 Pair<DialogType, Boolean>으로 묶어서 관리하는데 이렇게 해도 괜찮겠죠??

    private val _showDialog = MutableStateFlow(Pair(DialogType.LOGIN_GOOGLE, false))
    val showDialog: StateFlow<Pair<DialogType, Boolean>> get() = _showDialog

    fun showDialog(dialogType: DialogType, isVisible: Boolean) {
        _showDialog.update { it.copy(first = dialogType, second = isVisible) }
    }

@gaeulzzang gaeulzzang added ➕ [ADD] 부수적인 코드 추가 및 라이브러리 추가, 새로운 파일 생성 ♻️ [REFACTOR] 코드 리팩토링 🍂 가을 가응가 labels Jan 23, 2025
@gaeulzzang gaeulzzang self-assigned this Jan 23, 2025
@gaeulzzang gaeulzzang requested a review from a team as a code owner January 23, 2025 08:01
Copy link
Contributor

@twogarlic twogarlic left a comment

Choose a reason for hiding this comment

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

고생하셧숨니다ㅏㅏㅏ 왕깔꼼해서 쓸말이 업서요

Copy link
Member

@Eonji-sw Eonji-sw left a comment

Choose a reason for hiding this comment

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

난 내 다이얼로그가 이렇게 많이 쓰일줄 몰랐지~

…into feature/#38-appointment-dialog-add

# Conflicts:
#	presentation/src/main/java/com/sopt/presentation/mypage/MyPageRoute.kt
#	presentation/src/main/java/com/sopt/presentation/mypage/navigation/MyPageNavigation.kt
#	presentation/src/main/res/values/strings.xml
@gaeulzzang gaeulzzang merged commit eee16ac into develop Jan 24, 2025
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
➕ [ADD] 부수적인 코드 추가 및 라이브러리 추가, 새로운 파일 생성 🍂 가을 가응가 ♻️ [REFACTOR] 코드 리팩토링
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[ADD] : 다이얼로그 추가
3 participants