Skip to content

[#186] 검색 후 컨텐츠가 내비게이션 된 후, 다시 원래로 돌아오면 검색을 다시 하는 현상을 해결한다#197

Merged
opficdev merged 1 commit intodevelopfrom
fix/#186-navigation-search-again
Mar 6, 2026
Merged

[#186] 검색 후 컨텐츠가 내비게이션 된 후, 다시 원래로 돌아오면 검색을 다시 하는 현상을 해결한다#197
opficdev merged 1 commit intodevelopfrom
fix/#186-navigation-search-again

Conversation

@opficdev
Copy link
Copy Markdown
Owner

@opficdev opficdev commented Mar 6, 2026

@opficdev opficdev self-assigned this Mar 6, 2026
@gemini-code-assist
Copy link
Copy Markdown
Contributor

Summary of Changes

Hello, I'm Gemini Code Assist1! I'm currently reviewing this pull request and will post my feedback shortly. In the meantime, here's a summary to help you and other reviewers quickly get up to speed!

이 PR은 검색 후 콘텐츠 탐색을 거쳐 다시 검색 화면으로 돌아왔을 때, 동일한 검색 조건임에도 불구하고 검색이 다시 실행되는 현상을 해결합니다. 검색 쿼리나 텍스트가 실제로 변경되지 않았다면 불필요한 로직 실행을 방지하여 성능을 최적화하고 사용자 경험을 개선합니다.

Highlights

  • 검색 쿼리 중복 방지: SearchViewModel에서 검색 쿼리가 변경되지 않았을 경우 불필요한 검색 로직 재실행을 방지하는 가드 문을 추가했습니다.
  • 검색 텍스트 중복 방지: TodoListViewModel에서 검색 텍스트가 변경되지 않았을 경우 불필요한 검색 로직 재실행을 방지하는 가드 문을 추가했습니다.
Changelog
  • DevLog/Presentation/ViewModel/SearchViewModel.swift
    • setSearchQuery 액션 처리 시, 현재 검색 쿼리와 새 쿼리가 동일하면 로직 실행을 중단하는 가드 문을 추가했습니다.
  • DevLog/Presentation/ViewModel/TodoListViewModel.swift
    • setSearchText 액션 처리 시, 현재 검색 텍스트와 새 텍스트가 동일하면 로직 실행을 중단하는 가드 문을 추가했습니다.
Activity
  • 이 PR은 이슈 #186을 해결하고 닫습니다.
Using Gemini Code Assist

The full guide for Gemini Code Assist can be found on our documentation page, here are some quick tips.

Invoking Gemini

You can request assistance from Gemini at any point by creating a comment using either /gemini <command> or @gemini-code-assist <command>. Below is a summary of the supported commands on the current page.

Feature Command Description
Code Review /gemini review Performs a code review for the current pull request in its current state.
Pull Request Summary /gemini summary Provides a summary of the current pull request in its current state.
Comment @gemini-code-assist Responds in comments when explicitly tagged, both in pull request comments and review comments.
Help /gemini help Displays a list of available commands.

Customization

To customize Gemini Code Assist for GitHub experience, repository maintainers can create a configuration file and/or provide a custom code review style guide (such as PEP-8 for Python) by creating and adding files to a .gemini/ folder in the base of the repository. Detailed instructions can be found here.

Limitations & Feedback

Gemini Code Assist may make mistakes. Please leave feedback on any instances where its feedback is incorrect or counter productive. You can react with 👍 and 👎 on @gemini-code-assist comments. If you're interested in giving your feedback about your experience with Gemini Code Assist for Github and other Google products, sign up here.

You can also get AI-powered code generation, chat, as well as code reviews directly in the IDE at no cost with the Gemini Code Assist IDE Extension.

Footnotes

  1. Review the Privacy Notices, Generative AI Prohibited Use Policy, Terms of Service, and learn how to configure Gemini Code Assist in GitHub here. Gemini can make mistakes, so double check it and use code with caution.

Copy link
Copy Markdown
Contributor

@gemini-code-assist gemini-code-assist Bot left a comment

Choose a reason for hiding this comment

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

Code Review

이번 PR은 검색 후 다른 화면으로 이동했다가 돌아올 때 불필요하게 검색이 다시 실행되는 현상을 수정합니다. SearchViewModelTodoListViewModel에 각각 검색어가 동일할 경우 중복 실행을 방지하는 로직을 추가하셨습니다. 이 변경으로 인해 불필요한 API 호출 및 UI 업데이트를 막아 성능을 개선하고 사용자 경험을 향상시킬 수 있습니다. 변경된 코드는 명확하고 효과적으로 문제를 해결하고 있습니다.

@opficdev opficdev merged commit 76ad622 into develop Mar 6, 2026
1 check passed
@opficdev opficdev deleted the fix/#186-navigation-search-again branch March 6, 2026 02:58
@opficdev opficdev restored the fix/#186-navigation-search-again branch March 6, 2026 05:05
@opficdev opficdev deleted the fix/#186-navigation-search-again branch March 7, 2026 03:14
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.

검색 후 컨텐츠가 내비게이션 된 후, 다시 원래로 돌아오면 검색을 다시 하는 현상을 해결한다

1 participant