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

[hotfix] 신규단체개설뷰 이슈 해결 #252

Merged
merged 8 commits into from
Mar 13, 2024
Merged

Conversation

jihyunniiii
Copy link
Collaborator

Related issue 🛠

Work Description ✏️

  • 신규단체개설뷰 이슈를 해결하였습니다.
    • 중복확인 버튼 활성화/비활성화 로직 관리
    • 중복확인 버튼 클릭 시 단체 이름 앞 뒤 공백제거
    • 신규단체개설뷰 진입 시 앱 종료되는 문제 해결

Screenshot 📸

Screen_recording_20240313_152655.mp4

Uncompleted Tasks 😅

  • N/A

To Reviewers 📢

ㅠㅠ핑

Copy link
Member

@Doreminwoo Doreminwoo left a comment

Choose a reason for hiding this comment

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

제 할일 대신해주셔서 감사합니다!

@@ -48,13 +48,12 @@ class NewGroupKeywordFragment :
}

private fun collectData() {
newGroupViewModel.newGroupKeywordsState.flowWithLifecycle(lifecycle).onEach { uiState ->
newGroupViewModel.newGroupKeywordsState.flowWithLifecycle(viewLifecycleOwner.lifecycle).onEach { uiState ->
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
Collaborator Author

Choose a reason for hiding this comment

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

ㅋㅋ 이런 건 빨리 리팩하라핑

val newGroupKeywordName = MutableStateFlow<String>("")
val newGroupKeywordValue = MutableStateFlow<String>("")

val isNewGroupBtnEnabled: StateFlow<Boolean> = listOf(
currentPage,
newGroupName,
newGroupEmail,
isNewGroupBtnCheckName,
isGroupNameDuplicatedCheck,
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
Collaborator Author

Choose a reason for hiding this comment

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

아 ㅋㅋ 오빠가 만들어놓은 거랑 다른 목적으로 사용하려고 네이밍 수정했슴다 ㅋㅋ ㅜㅜ

@jihyunniiii jihyunniiii merged commit b463eb0 into develop Mar 13, 2024
1 check passed
@jihyunniiii jihyunniiii deleted the hotfix-new-group branch March 13, 2024 07:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

[hotfix] 신규단체개설뷰 이슈 해결
2 participants