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/#24] 모임 생성 구현 #34

Merged
merged 6 commits into from
Jan 13, 2024
Merged

[FEAT/#24] 모임 생성 구현 #34

merged 6 commits into from
Jan 13, 2024

Conversation

Mnseo
Copy link
Collaborator

@Mnseo Mnseo commented Jan 11, 2024

📌 개요

✨ 작업 내용

  • 모임 이미지 10MB 초과시 SnackBar 연결
  • TmSnackBar 추가(:Base)
  • Navigation Backstack 수정

✨ PR 포인트

Navigation 뒤로 가기시

무한 스크린 반복 & Navi 사라짐
문제 해결했습니다 테스트 완료했으나 혹시 모르니 확인해주세요

@kkk5474096 이거 navigation이 fragment 단위로 연결되어있어서 뒤로가기를 fragment에서 사용한 앞 화면으로 이동하는 함수를 연결하려고 TmScaffold 파라미터에 다 onClick을 넣어놨어요

        binding.composeMoim.setContent {
            val screenState by viewModel.screenState.collectAsState()
            when (screenState) {
                ScreenState.Topic -> MoimCreateTopic(viewModel) { goFrontScreen() }
                ScreenState.Name -> MoimCreateName(viewModel) { goFrontScreen() }
                ScreenState.Introduce -> MoimIntroduce(viewModel) { goFrontScreen()}
                ScreenState.DateTime -> MoimDateTime(viewModel) { goFrontScreen()}
                ScreenState.Address -> MoimAddress(viewModel) { goFrontScreen()}
                ScreenState.People -> MoimPeople(viewModel) { goFrontScreen()}
                else -> {}
            }
        }
        requireActivity().onBackPressedDispatcher.addCallback(viewLifecycleOwner, callback)
    }

모양이 매우 마음에 안드는데 이런식으로 하는게 적절하나요 ,,^^

📸 스크린샷/동영상

@Mnseo Mnseo added 민서 ADD feat 이외의 부수적인 코드, 파일, 라이브러리 추가 CHORE 버전 코드, 패키지 구조, 함수 및 변수명 변경 등의 작은 작업 labels Jan 11, 2024
@Mnseo Mnseo self-assigned this Jan 11, 2024
Copy link
Collaborator

@kkk5474096 kkk5474096 left a comment

Choose a reason for hiding this comment

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

모양은 맘에 안드셔도 쩔수없을거같네여,, 컴포즈와 같이 쓸라면 @Mnseo

@Mnseo Mnseo merged commit 473025a into main Jan 13, 2024
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ADD feat 이외의 부수적인 코드, 파일, 라이브러리 추가 CHORE 버전 코드, 패키지 구조, 함수 및 변수명 변경 등의 작은 작업 민서
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants