#80 소셜 검색 기능 #82
Open
#80 소셜 검색 기능 #82
Conversation
This file contains 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
Open
Sonny-Kor
force-pushed
the
#80-social-search
branch
2 times, most recently
from
163d3f2 to
f9cbbab
Compare
Category로 검색하는 부분이 삭제되어 Repo와 Usecase도 함께 변경
키워드 관련 업데이트
backIndicator, backButtonAppearce 로 변경
Sonny-Kor
force-pushed
the
#80-social-search
branch
from
0b167be to
0d2028f
Compare
Kyxxn
approved these changes
Jan 25, 2025
| } | ||
| }.inObjectScope(.container) |
There was a problem hiding this comment.
여기가 Repository의 싱글톤을 지원하는 코드가 맞는 거죠 ?
Comment on lines
+5
to
7
| func searchPost(keyword: String, category: PostCategory?) async throws -> [Post]? | ||
| func togglePostLike(postID: Post.ID) async throws -> Result<Post, Error> | ||
| func bringUserRoutine(routine: Routine) async throws -> Routine |
There was a problem hiding this comment.
여기도 추후에 throws 키워드가 삭제되어야 할 부분인 거죠 ?
Comment on lines
+85
to
+86
| input | ||
| .sink { [weak self] event in |
There was a problem hiding this comment.
저도 오늘 작업하면서 느꼈지만, Input도 결국 많아지면 transform 메소드가 안 예쁘네요
다닥다닥 붙어 있어서 오히려 가독성이 안 좋아보인다라는 느낌인데 어쩔 수 없는 거겠죠 ?
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.
#️⃣ 연관된 이슈
📝 작업 내용
📒 리뷰 노트
최근 검색어를 UserDefault로 Stroage에 저장하는 로직을 작성했습니다.
인기 검색어는 우선 고정으로 생각하고 작성하였습니다.
@Kyxxn 님, UserDefault에 저장하는 로직인데, 현재 Category와 저장하는 로직이 공통적으로 비슷 한데, 따로 UserDefault를 관리하는 객체를 만드는건 어떨까요?
키보드가 올라오는 속도에서 MainThread가 멈추는 현상이 간혹 발생하는데, 추후에 분석해서 수정해보겠습니다.
UISearchBar를 NavigationBar의 Title에 사용하니 높이가 56으로 변하네요,,
기존 NavigationBar의 높이는 44인지라 NavigationPush를 했을 때 조금 이동이 어색한 상황입니다.. (해결 방법을 아시는분들이 계실까요? 강제로 44로 줄이는 방법으로는 해결이 안되네요)
📸 스크린샷