[#595] SearchView에서 '#' 만 입력했을 때 보여주는 뷰를 다르게 구성한다#643
Merged
Conversation
Contributor
There was a problem hiding this comment.
Code Review
이번 풀이 리퀘스트는 사용자가 검색창에 '#' 문자만 입력했을 때 할 일 번호 검색 안내 화면을 보여주는 기능을 추가합니다. 이를 위해 다국어 문자열을 추가하고, SearchFeature 내에 안내 상태 판단 로직 및 검색 방지 처리를 구현하였으며, SearchView에 안내 뷰를 추가하고 관련 테스트를 작성했습니다. 리뷰어는 테스트 더블(SearchFeatureTestDoubles.swift)에서 '#' 문자열을 직접 비교하는 대신 SearchFeature.showsTodoNumberSearchInstruction(_:) 헬퍼 메서드를 활용하고 조건문을 병합하여 코드의 유지보수성과 가독성을 높이도록 제안했습니다.
Important
The consumer version of Gemini Code Assist on GitHub is being sunset. Starting June 18, 2026, new organization installations will be blocked, and all code review activity will officially cease on July 17, 2026.
For more details on the timeline and next steps, please review the Help Documentation.
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
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.
🔗 연관된 이슈
🎯 의도
#만 입력한 상태를 일반 검색 결과로 처리하지 않고, Todo 번호 검색을 이어서 입력하도록 안내하는 상태로 분리📝 작업 내용
📌 요약
#단독 입력 시 Todo/WebPage 조회를 시작하지 않고, Todo 번호 입력 안내 문구를 표시하도록 SearchView 검색 상태 개선🔍 상세
#단독 입력 상태를 판별하는showsTodoNumberSearchInstruction상태 추가#단독 입력 시 기존 검색 결과를 비우고 진행 중인 검색 요청을 취소하도록 처리#단독 입력 상태일 때 안내 뷰 표시Todo 번호를 입력해주세요#123처럼 입력하면 해당 Todo를 바로 찾을 수 있습니다.#단독 입력 시 조회가 시작되지 않는 회귀 테스트 추가📸 영상 / 이미지 (Optional)