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/home search: 홈화면 검색 필터 구현 완료 #29

Merged
merged 60 commits into from
Nov 19, 2023

Conversation

mjkim1019
Copy link
Collaborator

Summary

Feature/home search: 홈화면 검색 필터 구현 완료

Description

  • FilterSearch -> Search, Search -> FilterSearch 이동 구현
  • filter는 navigation argument로 처리 (hiltViewModel 사용하여 인스턴스 동일하지 않음)
  • expanded card 구현
  • calendar (core>designsystem)에 구현
  • localDate formatting을 위해 core>util 모듈 구현 후 LocalDateAdapter 구현

관계성
SearchViewModel : SearchScreen, SearchFilterScreen
HomeViewModel : HomeScreen

어려웠던 점

  1. Calendar 직접 구현
  2. collectAsStateWithLifecycle() 정의했음에도 불구하고, ui 업데이트 자동으로 안되는 문제
    reference type 내부 값 변경 시에는 copy하여 instance 복사하여 새로운 instance 만들고 적용해야, stateflow가 값 인식
  3. navigation argument로 data class 타입 전달
    gson으로 직렬화하여 string 타입으로 전달하도록 함
    그런데 localDate는 자동으로 바뀌지 않음 -> LocalDateAdapter Gson으로 바뀌게 구현

Related Issue

#22

Sceenshot(option)

MutableStateFlow를 사용할 때 Filter 객체를 직접 수정하는 대신 copy 메서드를 사용하여 새로운 인스턴스를 생성해야함!!
@kang9366 kang9366 self-requested a review November 19, 2023 12:23
Copy link
Member

@kang9366 kang9366 left a comment

Choose a reason for hiding this comment

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

수고하셨습니당~

@mjkim1019 mjkim1019 merged commit bfcfb69 into develop Nov 19, 2023
1 check passed
@mjkim1019 mjkim1019 deleted the feature/home_search branch November 19, 2023 12:28
@mjkim1019
Copy link
Collaborator Author

close #22

@kang9366 kang9366 linked an issue Feb 21, 2024 that may be closed by this pull request
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

feature/home_search: 검색 dialog 필터
2 participants