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

[Fix/#104] 다크모드&라이트 모드 설정 #105

Merged
merged 6 commits into from
Jan 27, 2024
Merged

Conversation

Mnseo
Copy link
Collaborator

@Mnseo Mnseo commented Jan 26, 2024

📌 개요

✨ 작업 내용

  • Setting: 라이트모드 / 다크 모드 자동 변경 설정
  • Setting: homeFragment에서만 뒤로가기 두번시 종료
  • Teum1-20: moim 화면 버튼 빼고 스크롤 하기 수정
  • Teum1-7 : 모임 생성 전체적인 플로우 오류 수정 + 화면 수정
  • Teum1-30 : 모임 상세 페이지 사진 크롭 수정

다른 프래그먼트에서는 navGraph 그대로 적용됩니다

✍🏻 메모

back stack 수정했습니다

@unam98 navGraph 손본 김에 말씀하셨던 문제를 해결하려고 했었는데 어제 제가 말한 방법으로 해결이 안됩니다 제 생각에는 navGraph에서 아예 teumteumFragment를 빼고 main에서 메뉴 클릭했을때

IF 온보딩 안했음 -> onboarding Activity / ELSE -> 그 뒤에 있는 화면

이렇게 이어줘야 할 것 같아요

@Mnseo Mnseo self-assigned this Jan 26, 2024
@Mnseo Mnseo added 민서 FIX 버그 및 오류 해결 UI UI 관련 작업 labels Jan 26, 2024
Comment on lines 26 to 34
data class Friends(
val friends: List<FriendMyPage>,
val introductions: List<Friend>
) : java.io.Serializable No newline at end of file
val introductions: List<Friend>?
) : java.io.Serializable

@Serializable
data class FriendRecommend(
val friends: List<FriendMyPage>
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

@unam98 이쪽 introduction이 있어서 친구 조회가 안돼서 수정했습니다 참고해주세요

Copy link
Member

@blueme0 blueme0 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 merged commit 8f0a7a8 into main Jan 27, 2024
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
FIX 버그 및 오류 해결 UI UI 관련 작업 민서
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants