-
Notifications
You must be signed in to change notification settings - Fork 3
책 검색 화면 UI, 검색 결과 출력 및 아이템 선택 동작 구현 #49
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
Merged
JangMuho
merged 33 commits into
boostcampwm2025:dev
from
JangMuho:feature/book-search-screen
Jan 7, 2026
Merged
책 검색 화면 UI, 검색 결과 출력 및 아이템 선택 동작 구현 #49
JangMuho
merged 33 commits into
boostcampwm2025:dev
from
JangMuho:feature/book-search-screen
Jan 7, 2026
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
- UiModel로 변경할 때 하나의 String으로 변환할 예정
- NaverBookItem을 BookUIModel로 변환하는 로직을 별도의 매퍼 파일로 분리 - 불필요한 ImmutableList import 제거
…rchNavGraph 수정 - NavController를 직접 참조하는 대신 MainNavigator를 통해 화면 이동을 처리하도록 변경
- ViewModel의 개행 정리 - 불필요한 Import 정리
- snapshotFlow로 Screen에서의 텍스트 변경을 관찰하여 uiState.query와 동기화
…o feature/book-search-screen
- 문자열로 바꿀 경우 ", "도 검색될 수 있기에 리스트 형태로 변경
- UI에서 Text로 표기할 때만 joinToString(", ")을 사용
- Column의 Modifier.padding(start = ~) 대신 상위 Row의 spacedBy로 변경
Closed
7 tasks
- 파라미터 공백 정리 - 한 줄로 나열된 코드 개행 정리
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
#️⃣연관된 이슈
📝작업 내용
🤔시행착오
@HiltViewModel을 쓰는 Screen을 표시하기 위해서는@HiltAndroidApp과@AndroidEntryPoint가 필요했습니다.@SerialName("")에 불필요한 공백을 추가해버리는 실수로 인해 앱이 크래시되는 문제가 있었습니다.💬리뷰 요구사항
🖼️동영상 및 스크린샷
MainActivity의 진입점을MainApp(navigator)으로 수정하고MainNaviagtor내부의startDestination을Route.BookSearch로 수정하면 동작을 확인하실 수 있습니다.book_search.mp4
save_error.mp4
정렬 기준이 문제인가 싶어 네이버 API 문서를 다시 확인해보니 기본값이 정확도순이었습니다...
대신 영어로 검색하면 잘 나옵니다.